diff options
author | Donald Sharp <sharpd@nvidia.com> | 2021-10-04 15:28:36 +0200 |
---|---|---|
committer | Donald Sharp <sharpd@nvidia.com> | 2021-10-04 15:28:36 +0200 |
commit | c733548963447f4383866f66117fd5e302d53baa (patch) | |
tree | 2320fb7eef4d8adf4dc097f4808d107953f557e6 /ripngd/ripng_interface.c | |
parent | Merge pull request #9714 from idryzhov/bgp-ext-comm-doc (diff) | |
download | frr-c733548963447f4383866f66117fd5e302d53baa.tar.xz frr-c733548963447f4383866f66117fd5e302d53baa.zip |
ripngd: Ensure better `struct thread *` semantics
1) Remove `struct thread *` pointers that are never used
2) Do not explicitly set the thread pointer to NULL.
FRR should only ever use the appropriate THREAD_ON/THREAD_OFF
semantics. This is espacially true for the functions we
end up calling the thread for.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to 'ripngd/ripng_interface.c')
-rw-r--r-- | ripngd/ripng_interface.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ripngd/ripng_interface.c b/ripngd/ripng_interface.c index f374fcb83..7b5e7604d 100644 --- a/ripngd/ripng_interface.c +++ b/ripngd/ripng_interface.c @@ -618,7 +618,6 @@ static int ripng_interface_wakeup(struct thread *t) ifp = THREAD_ARG(t); ri = ifp->info; - ri->t_wakeup = NULL; /* Join to multicast group. */ if (ripng_multicast_join(ifp, ri->ripng->sock) < 0) { |