diff options
author | Guy Harris <guy@alum.mit.edu> | 2017-03-02 15:02:14 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2017-03-02 23:02:40 +0000 |
commit | 4bd3c4d44ddcdf8e98fdf08a425e3a68e9b18395 (patch) | |
tree | e5821754c923dc5100247df9b921e1fe98c31385 /wiretap | |
parent | b98bb5188c9082a06a370483c9474a929e957659 (diff) |
Remove a cast that now causes rather than squelching warnings.
We're now comparing an unsigned with an expression made mostly of
unsigned, so there's no need to cast the expression to long to squelch
signed vs. unsigned warnings.
Change-Id: I3b8c6f6faf26a9c252eb55d9e69fb298a3ad4c3b
Reviewed-on: https://code.wireshark.org/review/20347
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wiretap')
-rw-r--r-- | wiretap/k12.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/wiretap/k12.c b/wiretap/k12.c index 71c07a19eb..5196a0e1a6 100644 --- a/wiretap/k12.c +++ b/wiretap/k12.c @@ -646,7 +646,7 @@ process_packet_data(struct wtap_pkthdr *phdr, Buffer *target, guint8 *buffer, switch(src_desc->input_type) { case K12_PORT_ATMPVC: - if ((long)(buffer_offset + length + K12_PACKET_OFFSET_CID) < record_len) { + if (buffer_offset + length + K12_PACKET_OFFSET_CID < record_len) { phdr->pseudo_header.k12.input_info.atm.vp = pntoh16(buffer + buffer_offset + length + K12_PACKET_OFFSET_VP); phdr->pseudo_header.k12.input_info.atm.vc = pntoh16(buffer + buffer_offset + length + K12_PACKET_OFFSET_VC); phdr->pseudo_header.k12.input_info.atm.cid = *((unsigned char*)(buffer + buffer_offset + length + K12_PACKET_OFFSET_CID)); |