Wireshark · Wireshark-commits: [Wireshark-commits] master 6a45dcd: dfilter: require spaces as set element separator

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=6a45dcd7a2ab695aade66499cdb61406a2196429
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

6a45dcd by Peter Wu (peter@xxxxxxxxxxxxx):

    dfilter: require spaces as set element separator
    
    Previously a filter such as `http.request.method in {"GET"HEAD""}` would
    be parsed as three strings (GET, HEAD and an empty string). As it seems
    more likely that people make typos rather than intending to construct
    such a filter, forbid this by always requiring a whitespace separator.
    
    Change-Id: I77e531fd6be072f62dd06aac27f856106c8920c6
    Reported-by: Stig Bjørlykke
    Reviewed-on: https://code.wireshark.org/review/26989
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  699ee5d   dfilter: make spaces around ".." optional in display filter
    adds  6a45dcd   dfilter: require spaces as set element separator


Summary of changes:
 epan/dfilter/dfilter-int.h |  1 +
 epan/dfilter/dfilter.c     |  1 +
 epan/dfilter/grammar.lemon |  4 ++--
 epan/dfilter/scanner.l     | 23 +++++++++++++++++++----
 4 files changed, 23 insertions(+), 6 deletions(-)