Wireshark-commits: [Wireshark-commits] master 701be1f: RTP player: don't crash when drawing channel
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, 2 Mar 2015 07:00:52 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=701be1f052922aa00288a8c28c42ee358587b7fb
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

701be1f by Jeff Morriss (jeff.morriss.ws@xxxxxxxxx):

    RTP player: don't crash when drawing channel graphs for very long calls.
    
    X11 can't handle pixmaps more than 32k pixels wide so don't try to feed it
    one.  This avoids crashes with the complaint "BadAlloc (insufficient
    resources for operation)".  Instead we simply truncate the graph (and tell the
    user).  Do this in a way that shows the user the empty portion of the graph.
    
    Hopefully the Qt port can fix this properly.
    
    Note: the bug report says that MacOS is not affected but it should be.  So
    apply this fix for all on-Windows systems.
    
    Bug: 2630
    Change-Id: I71e1bd2f9a62792db06ce887e2bbe7a96d110e0a
    Reviewed-on: https://code.wireshark.org/review/7464
    Petri-Dish: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  f9f7ff2   Don't read aliases from the personal hosts file.
    adds  701be1f   RTP player: don't crash when drawing channel graphs for very long calls.


Summary of changes:
 ui/gtk/rtp_player.c |   28 ++++++++++++++++++++++++++--
 1 file changed, 26 insertions(+), 2 deletions(-)