diff options
author | Dario Lombardo <lomato@gmail.com> | 2016-06-28 10:43:12 +0200 |
---|---|---|
committer | Dario Lombardo <lomato@gmail.com> | 2016-06-28 14:27:09 +0000 |
commit | 670b1199e3c3975755d4c717d8d991c1d56b597c (patch) | |
tree | ba2147c2eb05c0f70449696b9dcce9bbadb90df9 | |
parent | 8c006feeab858286897879e0c14f0573b4521ea6 (diff) |
print.c NULL pointer check in print_escaped
Added NULL pointer check in print_escaped functions.
Requested by comment in change 16034.
Change-Id: Id172d772d9b5cb4bcd31fe8b42286f885ec5b968
Reviewed-on: https://code.wireshark.org/review/16188
Petri-Dish: Dario Lombardo <lomato@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Dario Lombardo <lomato@gmail.com>
-rw-r--r-- | epan/print.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/epan/print.c b/epan/print.c index f981659f45..cf7fa148ed 100644 --- a/epan/print.c +++ b/epan/print.c @@ -1389,6 +1389,10 @@ print_escaped_xml(FILE *fh, const char *unescaped_string) const char *p; char temp_str[8]; + if (fh == NULL || unescaped_string == NULL) { + return; + } + for (p = unescaped_string; *p != '\0'; p++) { switch (*p) { case '&': @@ -1423,6 +1427,10 @@ print_escaped_bare(FILE *fh, const char *unescaped_string, gboolean change_dot) const char *p; char temp_str[8]; + if (fh == NULL || unescaped_string == NULL) { + return; + } + for (p = unescaped_string; *p != '\0'; p++) { switch (*p) { case '"': |