buglwIP - A Lightweight TCP/IP stack - Bugs: bug #19699, Bug in SNMP ASN1 decode; submitted...

 
 

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

bug #19699: Bug in SNMP ASN1 decode; submitted patch

Submitted by:  Jared Momose <jmomose>
Submitted on:  Wed 25 Apr 2007 07:27:22 PM UTC  
 
Category:  UDP Severity:  2 - Minor
Item Group:  Faulty Behaviour Status:  Fixed
Privacy:  Public Assigned to:  Simon Goldschmidt <goldsimon>
Open/Closed:  Closed Planned Release:  None
lwIP version:  1.2.0

( Jump to the original submission)

Mon 20 Apr 2009 08:57:30 AM UTC, comment #6: 

I've opened a task for this: we don't know how the status is for SNMP on big-endian, but this bug is fixed.

task #9332: Test SNMP on big-endian

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
Sat 09 Jun 2007 10:40:18 AM UTC, comment #5: 

Matching patch item (http://savannah.nongnu.org/patch/?5896) is check in and closed...

Frédéric Bernon <fbernon>
Project Member
Mon 21 May 2007 04:42:46 PM UTC, comment #4: 

Me too. Although this looks like it needs some more attention...

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
Mon 21 May 2007 04:37:05 PM UTC, comment #3: 

Agree with Jonathan

Frédéric Bernon <fbernon>
Project Member
Fri 27 Apr 2007 12:43:01 AM UTC, comment #2: 

Wouldn't it be better to check the patch in now, but leave this bug report open so the opportunity for improvement isn't forgotten about.

Jonathan Larmour <jifl>
Project Member
Thu 26 Apr 2007 07:19:33 AM UTC, comment #1: 

This could well be a problem, only tested this on little-endian archs. Maybe there are more routines in there not big endian proof.

But I don't like the #ifdef "solution", I'd prefer a bit cleaner, proper C solution.

Need to think a bit more about that, or find someone to continue and improve my attempts.

Christiaan Simons <christiaans>
Project Member
Wed 25 Apr 2007 07:27:22 PM UTC, original submission:  

There is a bug in the SNMP ASN1 decode routine on Big Endian systems. Attached is a patch that fixes the problem.

Jared Momose <jmomose>

 

Attached Files
file #12597:  asn1_dec.c.patch added by jmomose (413B - text/plain - Patch file)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by kieranm (Updated the item)
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by fbernon (Posted a comment)
  • -email is unavailable- added by jifl (Posted a comment)
  • -email is unavailable- added by christiaans (Posted a comment)
  • -email is unavailable- added by jmomose (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

     

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-04-20 goldsimon StatusNone => Fixed
        Assigned toNone => goldsimon
        Open/ClosedOpen => Closed
    2007-08-29 kieranm Severity3 - Normal => 2 - Minor
    2007-04-25 jmomose Attached File- => Added asn1_dec.c.patch, #12597

    Back to the top


    Powered by Savane 3.5