Wireshark-commits: [Wireshark-commits] master-2.2 e23a7d0: Fix handling of parameters with no value
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 23 Aug 2016 02:50:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e23a7d02ff403906ece7452b23f4ec2f54a8af7a
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

e23a7d0 by Guy Harris (guy@xxxxxxxxxxxx):

    Fix handling of parameters with no value.
    
    Return a value length of 0 if there's no parameter value.
    
    (And don't return anything if the pointers through which we return them
    are null.)
    
    If no value is present, return NULL from ws_find_media_type_parameter().
    
    Change-Id: I32b57623d7651bcf065af5b81f2390a600988b21
    Reviewed-on: https://code.wireshark.org/review/17255
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit ff00166a96d887e03899577caae3a355c26b0837)
    Reviewed-on: https://code.wireshark.org/review/17256
    

Actions performed:

    from  1c167f8   Add cast to squelch "building with a C++ compiler" warnings.
    adds  e23a7d0   Fix handling of parameters with no value.


Summary of changes:
 epan/media_params.c |   15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)