diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2015-03-03 08:54:54 +0100 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2015-04-21 10:18:01 +0200 |
| commit | d43f8b39b075fe60e0c8fdb33b07b284d3fae503 (patch) | |
| tree | 8c98dce915aaaabd0707a12c2c08199d0bfdec2e /bgpd | |
| parent | b1672ce858cc9c16fd7cc67b673aa241d9583a59 (diff) | |
| download | quagga-d43f8b39b075fe60e0c8fdb33b07b284d3fae503.tar.bz2 quagga-d43f8b39b075fe60e0c8fdb33b07b284d3fae503.tar.xz | |
bgpd, zebra: fix struct/pointer sizeof mixups
Two places were taking sizeof(pointer) instead of the sizeof(struct),
while performing operations on the struct. Both are initialisation
functions; I guess we haven't seen fallout since they weren't critical.
Fix anyway.
[v2: fix mistake that actually broke bgpd RS workqueue init]
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'bgpd')
| -rw-r--r-- | bgpd/bgp_route.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c index e7357e54..cd2737c8 100644 --- a/bgpd/bgp_route.c +++ b/bgpd/bgp_route.c @@ -1666,9 +1666,10 @@ bgp_process_queue_init (void) bm->process_main_queue->spec.max_retries = 0; bm->process_main_queue->spec.hold = 50; - memcpy (bm->process_rsclient_queue, bm->process_main_queue, - sizeof (struct work_queue *)); bm->process_rsclient_queue->spec.workfunc = &bgp_process_rsclient; + bm->process_rsclient_queue->spec.del_item_data = &bgp_processq_del; + bm->process_rsclient_queue->spec.max_retries = 0; + bm->process_rsclient_queue->spec.hold = 50; } void |