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: UDPSeverity: 2 - Minor
Item Group: Faulty BehaviourStatus: Fixed
Privacy: PublicAssigned to: Simon Goldschmidt <goldsimon>
Open/Closed: ClosedPlanned Release: None
lwIP version: 1.2.0

(Jump to the original submission 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
  • -unavailable- added by kieranm (Updated the item)
  • -unavailable- added by goldsimon (Posted a comment)
  • -unavailable- added by fbernon (Posted a comment)
  • -unavailable- added by jifl (Posted a comment)
  • -unavailable- added by christiaans (Posted a comment)
  • -unavailable- added by jmomose (Submitted the item)
  •  

    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 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 20 Apr 2009 08:57:30 AM UTCgoldsimonStatusNone=>Fixed
      Assigned toNone=>goldsimon
      Open/ClosedOpen=>Closed
    Wed 29 Aug 2007 09:18:46 AM UTCkieranmSeverity3 - Normal=>2 - Minor
    Wed 25 Apr 2007 07:27:22 PM UTCjmomoseAttached File-=>Added asn1_dec.c.patch, #12597

    Back to the top


    Powered by Savane 3.1-cleanup1