diff options
| author | Nick Mathewson <nickm@torproject.org> | 2008-12-18 04:45:47 +0000 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2008-12-18 04:45:47 +0000 |
| commit | f3f6daf819b73516db0adb4b149a7849188b7038 (patch) | |
| tree | da29f60105e643dbc339d112d26202000481743f /src/or/dirserv.c | |
| parent | 676175103de747085d02a647fc200f77cb0f4260 (diff) | |
| download | tor-f3f6daf819b73516db0adb4b149a7849188b7038.tar tor-f3f6daf819b73516db0adb4b149a7849188b7038.tar.gz | |
Avoid nop call to control_event_or_authdir_new_descriptor that makes coverity think we are dereferencing a null pointer. It is safe, I think, but entirely too clever for our own good.
svn:r17668
Diffstat (limited to 'src/or/dirserv.c')
| -rw-r--r-- | src/or/dirserv.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/or/dirserv.c b/src/or/dirserv.c index b8665de77..336fa7c37 100644 --- a/src/or/dirserv.c +++ b/src/or/dirserv.c @@ -726,7 +726,8 @@ dirserv_add_descriptor(routerinfo_t *ri, const char **msg) tor_free(desc); } else { smartlist_t *changed; - control_event_or_authdir_new_descriptor("ACCEPTED", desc, desclen, *msg); + if (desc) + control_event_or_authdir_new_descriptor("ACCEPTED", desc, desclen, *msg); changed = smartlist_create(); smartlist_add(changed, ri); |