diff options
| author | ulfl <ulfl@f5534014-38df-0310-8fa8-9805f1628bb7> | 2005-03-25 17:24:48 +0000 |
|---|---|---|
| committer | ulfl <ulfl@f5534014-38df-0310-8fa8-9805f1628bb7> | 2005-03-25 17:24:48 +0000 |
| commit | 0079bed70ecff10597323b03712d9c1dbf4f7878 (patch) | |
| tree | c8f123449fc113ec3c42e9165080bc32c1344295 /epan/dissectors/packet-ppp.c | |
| parent | eb29f9b39dada4d98460e664741de200657caae9 (diff) | |
bugfix: similar problem than the previous one fixed by Gerald
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@13898 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-ppp.c')
| -rw-r--r-- | epan/dissectors/packet-ppp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/dissectors/packet-ppp.c b/epan/dissectors/packet-ppp.c index 4bb884d29d..62d45f8e62 100644 --- a/epan/dissectors/packet-ppp.c +++ b/epan/dissectors/packet-ppp.c @@ -2457,7 +2457,8 @@ dissect_bap_phone_delta_opt(const ip_tcp_opt *optp, tvbuff_t *tvb, offset += 2; length -= 2; - while (length > 0) { + /* XXX - Should we check for a maximum length instead of using a cast? */ + while (((gint) length) > 0) { subopt_type = tvb_get_guint8(tvb, offset); subopt_len = tvb_get_guint8(tvb, offset + 1); ti = proto_tree_add_text(field_tree, tvb, offset, subopt_len, |