diff options
author | Michael Mann <mmann78@netscape.net> | 2014-08-15 21:32:57 -0400 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2014-09-01 15:47:29 +0000 |
commit | b95e6e3a6195f7ae17a9c83cafbb1210c4aacc54 (patch) | |
tree | a885d7eeb7781ff9fe806a1c1dabd99454c0be5b | |
parent | f2a7a6d50378ff6000e8c777855df945165ca125 (diff) |
Escape double quotes when printing ABSOLUTE_TIME field
This regression was introduced by the commit e5353bf1198d6abf748de78084ff64f597f13663 done for bug 10081
Bug: 10213
Change-Id: I3925a47ef13055f10f49fa9fefd022731746fbb8
Reviewed-on: https://code.wireshark.org/review/2572
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Reviewed-by: Evan Huus <eapache@gmail.com>
Reviewed-by: Michael Mann <mmann78@netscape.net>
-rw-r--r-- | epan/ftypes/ftype-time.c | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/epan/ftypes/ftype-time.c b/epan/ftypes/ftype-time.c index 1bbf793169..8479a4c04b 100644 --- a/epan/ftypes/ftype-time.c +++ b/epan/ftypes/ftype-time.c @@ -324,28 +324,32 @@ value_get(fvalue_t *fv) } static int -absolute_val_repr_len(fvalue_t *fv, ftrepr_t rtype _U_) +absolute_val_repr_len(fvalue_t *fv, ftrepr_t rtype) { gchar *rep; - int ret; + int ret; rep = abs_time_to_str(NULL, &fv->value.time, ABSOLUTE_TIME_LOCAL, - rtype == FTREPR_DISPLAY); + rtype == FTREPR_DISPLAY); - ret = (int)strlen(rep) + 2; /* 2 for opening and closing quotes */ + ret = (int)strlen(rep) + ((rtype == FTREPR_DFILTER) ? 2 : 0); /* 2 for opening and closing quotes */ - wmem_free(NULL, rep); + wmem_free(NULL, rep); return ret; } static void -absolute_val_to_repr(fvalue_t *fv, ftrepr_t rtype _U_, char *buf) +absolute_val_to_repr(fvalue_t *fv, ftrepr_t rtype, char *buf) { gchar *rep = abs_time_to_str(NULL, &fv->value.time, ABSOLUTE_TIME_LOCAL, - rtype == FTREPR_DISPLAY); - sprintf(buf, "\"%s\"", rep); - wmem_free(NULL, rep); + rtype == FTREPR_DISPLAY); + if (rtype == FTREPR_DFILTER) { + sprintf(buf, "\"%s\"", rep); + } else { + strcpy(buf, rep); + } + wmem_free(NULL, rep); } static int |