diff options
author | Ben Pfaff <blp@nicira.com> | 2012-12-26 09:43:51 -0800 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2013-01-08 17:03:46 -0800 |
commit | bfe834eb8e712ac195c5b31839556a3a00f2b228 (patch) | |
tree | d8246d28594b62136a2930a8c8f7da385fe278b0 /ovsdb | |
parent | 5684eb749042325111e05f41b20258e6824fc0b5 (diff) |
ovsdb-server: Fix memory leak when manager table has no status column.
This is a corner case. I doubt that it's triggered in practice outside the
unit tests.
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Ethan Jackson <ethan@nicira.com>
Diffstat (limited to 'ovsdb')
-rw-r--r-- | ovsdb/ovsdb-server.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/ovsdb/ovsdb-server.c b/ovsdb/ovsdb-server.c index 69548c20..6032d732 100644 --- a/ovsdb/ovsdb-server.c +++ b/ovsdb/ovsdb-server.c @@ -523,6 +523,10 @@ write_string_string_column(struct ovsdb_row *row, const char *column_name, datum = get_datum(row, column_name, OVSDB_TYPE_STRING, OVSDB_TYPE_STRING, UINT_MAX); if (!datum) { + for (i = 0; i < n; i++) { + free(keys[i]); + free(values[i]); + } return; } |