Remove unnecessary test.
Also, from me: add a comment explaining why the test isn't being done.
Reviewed-by: Guy Harris <guy@alum.mit.edu>
diff --git a/CREDITS b/CREDITS
index 205af0f..9e7e875 100644
--- a/CREDITS
+++ b/CREDITS
@@ -127,6 +127,7 @@
Michael Shalayeff <mickey at openbsd dot org>
Michael Shields <shields at msrl dot com>
Michael T. Stolarchuk <mts at off dot to>
+ Michal Sekletar <msekleta at redhat dot com>
Michele "mydecay" Marchetto <smarchetto1 at tin dot it>
Mike Frysinger <vapier at gmail dot com>
Minto Jeyananth <minto at juniper dot net>
diff --git a/print-atm.c b/print-atm.c
index 135898b..935d182 100644
--- a/print-atm.c
+++ b/print-atm.c
@@ -270,10 +270,12 @@
printf(":%s ",
tok2str(msgtype2str, "msgtype#%d", p[MSG_TYPE_POS]));
- if (caplen < CALL_REF_POS+3) {
- printf("[|atm]");
- return;
- }
+ /*
+ * The call reference comes before the message type,
+ * so if we know we have the message type, which we
+ * do from the caplen test above, we also know we have
+ * the call reference.
+ */
call_ref = EXTRACT_24BITS(&p[CALL_REF_POS]);
printf("CALL_REF:0x%06x", call_ref);
} else {
