Skip to content
Snippets Groups Projects
  • Richard Mudgett's avatar
    bd28a9bb
    STUN/netsock2: Fix some valgrind uninitialized memory findings. · bd28a9bb
    Richard Mudgett authored
    * netsock2.c: Test the addr->len member first as it may be the only member
    initialized in the struct.
    
    * stun.c:ast_stun_handle_packet(): The combinded[] local array could get
    used uninitialized by ast_stun_request().  The uninitialized string gets
    copied to another location and could overflow the destination memory
    buffer.
    
    These valgrind findings were found for ASTERISK_27150 but are not
    necessarily a fix for the issue.
    
    Change-Id: I55f8687ba4ffc0f69578fd850af006a56cbc9a57
    bd28a9bb
    History
    STUN/netsock2: Fix some valgrind uninitialized memory findings.
    Richard Mudgett authored
    * netsock2.c: Test the addr->len member first as it may be the only member
    initialized in the struct.
    
    * stun.c:ast_stun_handle_packet(): The combinded[] local array could get
    used uninitialized by ast_stun_request().  The uninitialized string gets
    copied to another location and could overflow the destination memory
    buffer.
    
    These valgrind findings were found for ASTERISK_27150 but are not
    necessarily a fix for the issue.
    
    Change-Id: I55f8687ba4ffc0f69578fd850af006a56cbc9a57