diff options
| author | Aurimas Liutikas <aurimas@google.com> | 2016-05-10 21:16:39 +0000 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2016-05-10 21:16:39 +0000 |
| commit | bc83d2d10e7b143b2519203ea8efb61af6768d64 (patch) | |
| tree | a37f51b25543a43ad56bde1f5cda9076963837d0 | |
| parent | dbb0163636b74b580abcf7d1de8d834163ca0a3a (diff) | |
| parent | 51a752cf2fdf780884110135f1fa453745f47ea1 (diff) | |
| download | tcpdump-bc83d2d10e7b143b2519203ea8efb61af6768d64.tar.gz | |
Merge changes I8ae9ec5b,I71bd5c50,I450cca83,I30d77415 am: 37816ede8b
am: 51a752cf2f
* commit '51a752cf2fdf780884110135f1fa453745f47ea1':
Enable -Werror in external/tcpdump
Clean up version test.
Add a missing space to a printout.
ISOCLNS: Fix some warnings
Change-Id: I69b896e407776878df50b0ecf19c2e7f6b6f66bd
| -rw-r--r-- | Android.mk | 1 | ||||
| -rw-r--r-- | print-isoclns.c | 15 | ||||
| -rw-r--r-- | print-nflog.c | 2 |
3 files changed, 10 insertions, 8 deletions
@@ -164,6 +164,7 @@ include $(CLEAR_VARS) LOCAL_SRC_FILES := $(tcpdump_src_files) LOCAL_CFLAGS += -DHAVE_CONFIG_H LOCAL_CFLAGS += -D_U_="__attribute__((unused))" +LOCAL_CFLAGS += -Werror LOCAL_SHARED_LIBRARIES += libssl libcrypto LOCAL_STATIC_LIBRARIES += libpcap LOCAL_MODULE_PATH := $(TARGET_OUT_OPTIONAL_EXECUTABLES) diff --git a/print-isoclns.c b/print-isoclns.c index f8de7bce..4536f65b 100644 --- a/print-isoclns.c +++ b/print-isoclns.c @@ -559,8 +559,8 @@ struct isis_tlv_ptp_adj { uint8_t neighbor_extd_local_circuit_id[4]; }; -static void osi_print_cksum(netdissect_options *, const uint8_t *pptr, uint16_t checksum, - u_int checksum_offset, u_int length); +static void osi_print_cksum(netdissect_options *, const uint8_t *pptr, + uint16_t checksum, int checksum_offset, int length); static int clnp_print(netdissect_options *, const uint8_t *, u_int); static void esis_print(netdissect_options *, const uint8_t *, u_int); static int isis_print(netdissect_options *, const uint8_t *, u_int); @@ -3075,9 +3075,8 @@ isis_print(netdissect_options *ndo, } static void -osi_print_cksum(netdissect_options *ndo, - const uint8_t *pptr, uint16_t checksum, - u_int checksum_offset, u_int length) +osi_print_cksum(netdissect_options *ndo, const uint8_t *pptr, + uint16_t checksum, int checksum_offset, int length) { uint16_t calculated_checksum; @@ -3087,12 +3086,14 @@ osi_print_cksum(netdissect_options *ndo, * or the base pointer is not sane */ if (!checksum + || length < 0 + || checksum_offset < 0 || length > ndo->ndo_snaplen || checksum_offset > ndo->ndo_snaplen || checksum_offset > length) { - ND_PRINT((ndo, "(unverified)")); + ND_PRINT((ndo, " (unverified)")); } else { - unsigned char *truncated = "trunc"; + const char *truncated = "trunc"; #if 0 printf("\nosi_print_cksum: %p %u %u %u\n", pptr, checksum_offset, length, ndo->ndo_snaplen); ND_TCHECK2(pptr, checksum_offset+length); diff --git a/print-nflog.c b/print-nflog.c index 95da4cba..4234022c 100644 --- a/print-nflog.c +++ b/print-nflog.c @@ -81,7 +81,7 @@ nflog_if_print(netdissect_options *ndo, return h_size; } - if (!(hdr->nflog_version) == 0) { + if (hdr->nflog_version != 0) { ND_PRINT((ndo, "version %u (unknown)", hdr->nflog_version)); return h_size; } |