aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-http-urlencoded.c
diff options
context:
space:
mode:
authorJakub Zawadzki <darkjames-ws@darkjames.pl>2013-06-17 07:41:48 +0000
committerJakub Zawadzki <darkjames-ws@darkjames.pl>2013-06-17 07:41:48 +0000
commitf2f2f13db3b5b86f9dd96a54f8175f8576e7a10b (patch)
treeb6aafd5df086517c8dd5cacadfbb1cf0d71bd1a4 /epan/dissectors/packet-http-urlencoded.c
parent6dd022f14fb400d79972885d1e8ad3b40d79add7 (diff)
Fix for previous commit, don't use string if get_form_key_value() failed.
svn path=/trunk/; revision=49978
Diffstat (limited to 'epan/dissectors/packet-http-urlencoded.c')
-rw-r--r--epan/dissectors/packet-http-urlencoded.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-http-urlencoded.c b/epan/dissectors/packet-http-urlencoded.c
index 333f3677b1..8195113297 100644
--- a/epan/dissectors/packet-http-urlencoded.c
+++ b/epan/dissectors/packet-http-urlencoded.c
@@ -161,17 +161,17 @@ dissect_form_urlencoded(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
sub = proto_item_add_subtree(ti, ett_form_urlencoded);
next_offset = get_form_key_value(tvb, &key, offset, '=');
- proto_tree_add_string(sub, hf_form_key, tvb, offset, next_offset - offset, key);
if (next_offset == -1)
break;
+ proto_tree_add_string(sub, hf_form_key, tvb, offset, next_offset - offset, key);
proto_item_append_text(sub, ": \"%s\"", key);
offset = next_offset+1;
next_offset = get_form_key_value(tvb, &value, offset, '&');
- proto_tree_add_string(sub, hf_form_value, tvb, offset, next_offset - offset, value);
if (next_offset == -1)
break;
+ proto_tree_add_string(sub, hf_form_value, tvb, offset, next_offset - offset, value);
proto_item_append_text(sub, " = \"%s\"", value);
offset = next_offset+1;