Wireshark-commits: [Wireshark-commits] master e91af83: Replace some seeks forward with wtap_read_by
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 29 Sep 2016 04:36:11 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e91af83c637adaa7a9837de65f50596a32b08db0
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

e91af83 by Guy Harris (guy@xxxxxxxxxxxx):

    Replace some seeks forward with wtap_read_bytes() with a null buffer pointer.
    
    If the seek forward is just skipping record content that's not
    (currently) interesting, use wtap_read_bytes() with a null buffer
    pointer; it catches short "reads" and requires less seeking, so it may
    work better when reading from a pipe.
    
    Change-Id: Ifb07d20e0391a8ed97da85149d971b4e9ef093a8
    Reviewed-on: https://code.wireshark.org/review/17976
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  48b6415   PROFINET: Fix variable may not have been initialized warning
    adds  e91af83   Replace some seeks forward with wtap_read_bytes() with a null buffer pointer.


Summary of changes:
 wiretap/5views.c              |    2 +-
 wiretap/lanalyzer.c           |    3 ++-
 wiretap/libpcap.c             |    2 +-
 wiretap/mp2t.c                |    2 +-
 wiretap/nettl.c               |   16 +++++++++-------
 wiretap/network_instruments.c |   12 +++++++-----
 wiretap/netxray.c             |    2 +-
 wiretap/peektagged.c          |    5 +++--
 wiretap/radcom.c              |   22 +++++++++++++++-------
 wiretap/snoop.c               |    2 +-
 wiretap/vwr.c                 |    2 +-
 11 files changed, 42 insertions(+), 28 deletions(-)