diff options
| author |   Benoît Ganne <bganne@cisco.com> | 2020-04-16 12:47:47 +0200 |
|---|---|---|
| committer |   Neale Ranns <nranns@cisco.com> | 2020-04-16 17:15:03 +0000 |
| commit | bd7f3422bbe38ba87888b765e94b56bfcbb9602c (patch) | |
| tree | 4d67199d2495f060af0beb4c22cac9c610e7ce9c /src/plugins/igmp/igmp_config.c | |
| parent | 063f2b86edafdc17d285bebc7722cad19c885db1 (diff) | |
igmp: fix igmp proxy group merge
When merging proxy groups in igmp_proxy_device_merge_group(), the call
to igmp_proxy_device_merge_src() can end up removing the current proxy
group via igmp_group_clear(). When that happens, it must returns NULL so
that igmp_proxy_device_merge_config() does not send a IGMPv3 report for
a dead proxy group.
Make igmp_group_clear() reset the group pointer to NULL to fix this bug
and to detect similar bugs more easily.
Type: fix
Change-Id: I229e55b5bfa71734d7844893f5209a66fa3cc8ae
Signed-off-by: Benoît Ganne <bganne@cisco.com>
Diffstat (limited to 'src/plugins/igmp/igmp_config.c')
| -rw-r--r-- | src/plugins/igmp/igmp_config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/igmp/igmp_config.c b/src/plugins/igmp/igmp_config.c index f9acc299ff1..7637adba5bf 100644 --- a/src/plugins/igmp/igmp_config.c +++ b/src/plugins/igmp/igmp_config.c @@ -31,7 +31,7 @@ igmp_clear_config (igmp_config_t * config) /* *INDENT-OFF* */ FOR_EACH_GROUP (group, config, ({ - igmp_group_clear (group); + igmp_group_clear (&group); })); /* *INDENT-ON* */ |