Wireshark-commits: [Wireshark-commits] master-1.12 eade3f2: Fix escaping of strings in UATs.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 20 Jul 2015 18:02:32 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=eade3f26060337a40d83e01bb3911190f708247c
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

eade3f2 by Guy Harris (guy@xxxxxxxxxxxx):

    Fix escaping of strings in UATs.
    
    Not only must characters that aren't printable ASCII characters be
    escaped, backslashes must be escaped (as backslash is an escape
    introducer) and double-quotes must be escaped (as double-quotes
    encapsulate strings).
    
    When constructing a string to hand to uat_load_str(), escape pathnames,
    as they are likely to contain backslashes on Windows, could contain
    backslashes on UN*X, and could contain quotes on UN*X and possibly
    Windows.  (Arguably, we should escape all the string arguments
    
    Bug: 11372
    Change-Id: I26efd1d89eb3e4d15a4a066f1bcb13fe79904dff
    Reviewed-on: https://code.wireshark.org/review/9718
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  93e13ef   GTPv2: always set direction before dissecting PCO IE
    adds  eade3f2   Fix escaping of strings in UATs.


Summary of changes:
 epan/dissectors/packet-dtls.c |    4 +++-
 epan/dissectors/packet-ssl.c  |    4 +++-
 epan/uat.c                    |    6 +++---
 3 files changed, 9 insertions(+), 5 deletions(-)