diff options
| author | Lutz, Arnoldo <arnoldo.lutz.guevara@hpe.com> | 2016-06-13 16:06:48 +0000 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2016-06-24 14:21:35 -0700 |
| commit | b1048e6af164ff6946ade8118adfc5a0f2c980d5 (patch) | |
| tree | 46fc930e1c79e6ea27bec7206cd1188b8ccf64da /lib/ovsdb-map-op.c | |
| parent | a12e2a88d6727d06d4f2fc245dff0b5f053da490 (diff) | |
| download | openvswitch-b1048e6af164ff6946ade8118adfc5a0f2c980d5.tar.gz | |
ovsdb-idl: Fix issues detected in Partial Map Update feature
We found some issues affecting Partial Map Update feature included in
master branch. This patch fixes a memory leak due to lack of freeing datum
allocated in the process of requesting a change to a map. It also fix an
error produced when NDEBUG flag is not set that causes an assertion when
preparing the map to be changed.
Fix of a memory leak not freeing datums.
Change use of ovsdb_idl_read function when preparing changes to maps.
Signed-off-by: arnoldo.lutz.guevara@hpe.com <arnoldo.lutz.guevara@hpe.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'lib/ovsdb-map-op.c')
| -rw-r--r-- | lib/ovsdb-map-op.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/ovsdb-map-op.c b/lib/ovsdb-map-op.c index 58f43dcda..23fc5f526 100644 --- a/lib/ovsdb-map-op.c +++ b/lib/ovsdb-map-op.c @@ -57,6 +57,7 @@ map_op_destroy_datum(struct map_op *map_op, const struct ovsdb_type *type) } else { ovsdb_datum_destroy(map_op->datum, type); } + free(map_op->datum); map_op->datum = NULL; } |