buglwIP - A Lightweight TCP/IP stack - Bugs: bug #49266, bug #49264: Still Crash Might Occur

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #49266: bug #49264: Still Crash Might Occur

Submitted by:  Arpit Agarwal <arpit_iitg>
Submitted on:  Tue 04 Oct 2016 11:59:01 AM UTC  
 
Category: sockets/netconnSeverity: 3 - Normal
Item Group: Crash ErrorStatus: Fixed
Privacy: PublicAssigned to: None
Open/Closed: ClosedPlanned Release: None
lwIP version: git head

Wed 05 Oct 2016 05:46:26 AM UTC, comment #4:

Thank you for testing and reporting back

Dirk Ziegelmeier <dziegel>
Project Member
Wed 05 Oct 2016 03:49:58 AM UTC, comment #3:

The fix seems to work. There is no crash issues now.
Sorry for opening a new bug, I some how missed the part that I can comment on the open / closed bugs. Will take care in future.

Arpit Agarwal <arpit_iitg>
Tue 04 Oct 2016 07:53:56 PM UTC, comment #2:

Arpit,
when a bug is fixed and you have more comments, please do post to the old, closed, bug report instead of opening a new one.
You can do that by either entering the URL manually (if you know the bug number) or click on "Display Criteria" above the bug list and select "Any" in the open/closed drop down box.

Simon Goldschmidt <goldsimon>
Project Administrator
Tue 04 Oct 2016 12:07:36 PM UTC, comment #1:

Implemented a different approach. Please try if it works! Thank you!

Dirk Ziegelmeier <dziegel>
Project Member
Tue 04 Oct 2016 11:59:01 AM UTC, original submission:

The Fix for bug #49264 is still not proper as per patch:
http://git.savannah.gnu.org/cgit/lwip.git/commit/?id=30251b14046c942a03f088c0a7cc991c4f2e4e9b

Still the following line of code should be modified:

LWIP_ASSERT("socket has no netconn", sockets[s].conn != NULL);

as socktes[s] may not be proper when LWIP_SOCKET_OFFSET is used.

With

LWIP_ASSERT("socket has no netconn", sockets[sa-1].conn != NULL);

Arpit Agarwal <arpit_iitg>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by goldsimon (Posted a comment)
  • -unavailable- added by dziegel (Posted a comment)
  • -unavailable- added by arpit_iitg (Submitted the item)
  • -unavailable- added by arpit_iitg
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 05 Oct 2016 05:46:26 AM UTCdziegelStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Tue 04 Oct 2016 12:07:36 PM UTCdziegelStatusNone=>Ready For Test
    Tue 04 Oct 2016 11:59:01 AM UTCarpit_iitgCarbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1