CFM: There must be at least one byte of MA short name.

This should fix Coverity CID 1400555.

diff --git a/print-cfm.c b/print-cfm.c
index 09199fa580ae9973eb1d3c269c6801c834f85dbb..450922a5cd0a14ed296e26a2d63f282f69e6a2fb 100644 (file)
--- a/print-cfm.c
+++ b/print-cfm.c
@@ -368,8 +368,11 @@ cfm_print(netdissect_options *ndo,
                    md_nameformat,
                    md_namelength));
 
-            /* -2 for the MA short name format and length */
-            if (md_namelength > names_data_remaining - 2) {
+            /*
+             * -3 for the MA short name format and length and one byte
+             * of MA short name.
+             */
+            if (md_namelength > names_data_remaining - 3) {
                 ND_PRINT((ndo, " (too large, must be <= %u)", names_data_remaining - 2));
                 return;
             }