bugLiquid War 6 - Bugs: bug #41184, Segmentation fault in...

 
 

bug #41184: Segmentation fault in lw6net_if_guess_local() with tunnel interfaces

Submitter:  Bernard Cafarelli <voyageur>
Submitted:  Fri 10 Jan 2014 10:36:05 AM UTC
   
 
Category:  Engine Severity:  3 - Normal
Status:  Fixed Privacy:  Public
Assigned to:  ufoot Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 10 Jan 2014 12:15:39 PM UTC, comment #1: 

Thanks for this complete report. I've committed the fix in current GIT develop branch : 71ec55c94bd094430fe809c9391678720cf91b92

Have a nice day,

Christian.

Christian Mauduit <ufoot>
Group administrator
Fri 10 Jan 2014 10:36:05 AM UTC, original submission:  

In addition to standard eth0 interface, on my system I also have an openvpn tunnel interface tun0

This makes lw6net_if_guess_local() segfault (at startup):
Program received signal SIGSEGV, Segmentation fault.
0x0000000000512238 in lw6net_if_guess_local () at net-if.c:124
124   family = ifa->ifa_addr->sa_family;
(gdb) p ifa
$1 = (struct ifaddrs *) 0x9521f8
(gdb) p ifa->ifa_addr
$2 = (struct sockaddr *) 0x0
(gdb) p *ifa
$3 = {ifa_next = 0x9522b0, ifa_name = 0x95229c "tun0", ifa_flags = 69841,
  ifa_addr = 0x0, ifa_netmask = 0x0, ifa_ifu = {ifu_broadaddr = 0x0,
    ifu_dstaddr = 0x0}, ifa_data = 0x952814}

I modified the test to first check ifa_addr before the family itself, and the game now works fine

(tested On Gentoo amd64 with version 0.2.3551)

Bernard Cafarelli <voyageur>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ufoot (Posted a comment)
  • -email is unavailable- added by voyageur (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-05-09 ufoot StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2014-01-10 ufoot CategoryNone Engine
        StatusNone Ready For Test
        Assigned toNone ufoot
    2014-01-10 voyageur Attached File- Added liquidwar6-tun_interfaces.patch, #30242

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code