diff options
author | Guy Harris <guy@alum.mit.edu> | 2018-07-30 23:48:04 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2018-07-31 06:48:43 +0000 |
commit | 205a4ac1443d7f328768f80bbacbd7eb6beef93f (patch) | |
tree | 6ed3896bfd3bd930866f5902774a65caf6e988b2 | |
parent | 8f56eb5d00ec08b18e559d459566ca5fcbb53328 (diff) |
No need to initialize cursorname_len.
Not initializing it also means that we'll get an error from compilers
with sufficiently good dataflow analysis if we use it in, or before, the
call itself, which is a Good Thing as we *shouldn't* use it before we
know it.
Change-Id: I99aa3fedd2a04f5bb6e60e0f6f8b0a3682263351
Reviewed-on: https://code.wireshark.org/review/28888
Reviewed-by: Guy Harris <guy@alum.mit.edu>
-rw-r--r-- | epan/dissectors/packet-tds.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-tds.c b/epan/dissectors/packet-tds.c index e6a1b31165..bb8baa286e 100644 --- a/epan/dissectors/packet-tds.c +++ b/epan/dissectors/packet-tds.c @@ -2650,7 +2650,7 @@ dissect_tds5_curclose_token(tvbuff_t *tvb, packet_info *pinfo, guint offset, cur += 4; if (cursorid == 0) { - guint cursorname_len = 0; + guint cursorname_len; const guint8 *cursorname; proto_item *cursor_name_pi; |