diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2017-04-12 21:07:20 +0200 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2017-04-13 20:41:15 +0000 |
| commit | 3f71262bb1e7dd0b0fe062b6ec20f2cb2291255a (patch) | |
| tree | 38ce46b423efa9ccbf3976f071d0b6c3742b691f /epan/dissectors/packet-netlink-sock_diag.c | |
| parent | 0c597f111ffaacd12c1f1e46c0bfddcdfdec5b02 (diff) | |
netlink (sock_diag): Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang
The payload dissection should start after the header, so do not reset offset.
Fixes: v2.3.0rc0-3039-g92ebd63892 ("netlink: let subdissectors handle the netlink header")
Change-Id: I6bdfdd967bc56cb99c07f4f151ef96592e18a508
Reviewed-on: https://code.wireshark.org/review/21057
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-netlink-sock_diag.c')
| -rw-r--r-- | epan/dissectors/packet-netlink-sock_diag.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/epan/dissectors/packet-netlink-sock_diag.c b/epan/dissectors/packet-netlink-sock_diag.c index 1592072c9d..5000caee35 100644 --- a/epan/dissectors/packet-netlink-sock_diag.c +++ b/epan/dissectors/packet-netlink-sock_diag.c @@ -1126,8 +1126,6 @@ dissect_netlink_sock_diag(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, v info.pinfo = pinfo; info.data = data; - offset = 0; - switch (data->type) { case WS_TCPDIAG_GETSOCK: case WS_DCCPDIAG_GETSOCK: |