From d37b80a6d22f4af56d93602b9e41da710846bc70 Mon Sep 17 00:00:00 2001 From: Harald Welte Date: Thu, 15 Dec 2016 18:33:15 +0100 Subject: gtp.c: Improve debug statements about GTP header flags * as there are muptiple flags in the field, use plural * print the flags as hex value, not decimal. Hex is customary for bitfields. Change-Id: Ib23d80fae32b4e9fa146d82d8f5a1dada1a3cb2b --- gtp/gtp.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) (limited to 'gtp') diff --git a/gtp/gtp.c b/gtp/gtp.c index dcbe777..42aa4ea 100644 --- a/gtp/gtp.c +++ b/gtp/gtp.c @@ -210,7 +210,7 @@ static uint16_t get_seq(void *pack) } else if ((packet->flags & 0xe2) == 0x22) { /* Version 1 with seq */ return ntoh16(packet->gtp1l.h.seq); } else { - LOGP(DLGTP, LOGL_ERROR, "Unknown packet flag: %u\n", packet->flags); + LOGP(DLGTP, LOGL_ERROR, "Unknown packet flags: 0x%02x\n", packet->flags); return 0; } } @@ -246,7 +246,7 @@ static uint16_t get_hlen(void *pack) } else if ((packet->flags & 0xe7) == 0x20) { /* Short version 1 */ return GTP1_HEADER_SIZE_SHORT; } else { - LOGP(DLGTP, LOGL_ERROR, "Unknown packet flag: %u\n", packet->flags); + LOGP(DLGTP, LOGL_ERROR, "Unknown packet flags: 0x%02x\n", packet->flags); return 0; } } @@ -265,7 +265,7 @@ static uint32_t get_tei(void *pack) } else if ((packet->flags & 0xe0) == 0x20) { /* Version 1 */ return ntoh32(packet->gtp1l.h.tei); } else { - LOGP(DLGTP, LOGL_ERROR, "Unknown packet flag: %u\n", packet->flags); + LOGP(DLGTP, LOGL_ERROR, "Unknown packet flags: 0x%02x\n", packet->flags); return 0xffffffff; } } @@ -388,7 +388,7 @@ int gtp_req(struct gsn_t *gsn, int version, struct pdp_t *pdp, packet->gtp1l.h.tei = hton32(pdp->teic_gn); fd = gsn->fd1c; } else { - LOGP(DLGTP, LOGL_ERROR, "Unknown packet flag: %u\n", packet->flags); + LOGP(DLGTP, LOGL_ERROR, "Unknown packet flags: 0x%02x\n", packet->flags); return -1; } @@ -537,7 +537,7 @@ int gtp_resp(int version, struct gsn_t *gsn, struct pdp_t *pdp, else if (pdp) packet->gtp1l.h.tei = hton32(pdp->teic_gn); } else { - LOGP(DLGTP, LOGL_ERROR, "Unknown packet flag: %u\n", packet->flags); + LOGP(DLGTP, LOGL_ERROR, "Unknown packet flags: 0x%02x\n", packet->flags); return -1; } @@ -595,7 +595,7 @@ int gtp_notification(struct gsn_t *gsn, int version, packet->gtp1l.h.length = hton16(len - GTP1_HEADER_SIZE_SHORT); packet->gtp1l.h.seq = hton16(seq); } else { - LOGP(DLGTP, LOGL_ERROR, "Unknown packet flag: %u\n", packet->flags); + LOGP(DLGTP, LOGL_ERROR, "Unknown packet flags: 0x%02x\n", packet->flags); return -1; } @@ -2864,7 +2864,7 @@ int gtp_decaps1c(struct gsn_t *gsn) if (((pheader->flags & 0xf7) != 0x32)) { gsn->unsup++; GTP_LOGPKG(LOGL_ERROR, &peer, buffer, - status, "Unsupported packet flag\n"); + status, "Unsupported packet flags: 0x%02x\n", pheader->flags); continue; } @@ -3039,7 +3039,7 @@ int gtp_decaps1u(struct gsn_t *gsn) if (((pheader->flags & 0xf5) != 0x30)) { gsn->unsup++; GTP_LOGPKG(LOGL_ERROR, &peer, buffer, - status, "Unsupported packet flag\n"); + status, "Unsupported packet flags 0x%02x\n", pheader->flags); continue; } -- cgit v1.2.3