Fix an obvious case of incorrectly comparing the return value of tvb_length_remaining() against non-zero instead of greater than zero. There are other questionable uses of tvb_length_remaining() as well ... and maybe these should all be replaced with tvb_reported_length_remaining() as well?

#BACKPORT(1.10,1.8)

svn path=/trunk/; revision=50510

1 files changed, 3 insertions, 2 deletions

diff --git a/epan/dissectors/packet-slsk.c b/epan/dissectors/packet-slsk.c
index 8e923853df..72ce48ac82 100644
--- a/epan/dissectors/packet-slsk.c
+++ b/epan/dissectors/packet-slsk.c
@@ -261,8 +261,9 @@ static gboolean check_slsk_format(tvbuff_t *tvb, int offset, const char format[]
   }
 
   if (format[1] == '\0' ) {
-    if (tvb_length_remaining(tvb, offset) != 0) return FALSE;  /* Checks for additional bytes at the end */
-      return TRUE;
+    if (tvb_length_remaining(tvb, offset) > 0) /* Checks for additional bytes at the end */
+      return FALSE;
+    return TRUE;
   }
   return check_slsk_format(tvb, offset, &format[1]);