Wireshark-commits: [Wireshark-commits] master-2.4 ac8d875: The string part of a NULL string is zero

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 08 Jun 2018 21:06:24 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ac8d87512b1357802bdc6909f17ffd7781c14020
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.4
Repository: wireshark

Commits:

ac8d875 by Guy Harris (guy@xxxxxxxxxxxx):

    The string part of a NULL string is zero bytes long, not 4 bytes long.
    
    There aren't 4 bytes of 'N', 'U', 'L', and 'L' in the packet, so the
    length is *not* 4 bytes.
    
    Change-Id: I81331ef3f307dc65458da37e7d46e299eb7e727a
    Reviewed-on: https://code.wireshark.org/review/28149
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 09a7e8e863ac8a3b80a7a016b9943f815fc92367)
    Reviewed-on: https://code.wireshark.org/review/28151
    

Actions performed:

    from  bf9055f   Debian: GNUTLS updates.
    adds  ac8d875   The string part of a NULL string is zero bytes long, not 4 bytes long.


Summary of changes:
 epan/dissectors/packet-mysql.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)