lwIP - A Lightweight TCP/IP stack - Patches: patch #6215, ifAdminStatus write support for...
You are not allowed to post comments on this tracker with your current authentication level.
patch #6215: ifAdminStatus write support for SNMP
Submitter: | Artem Migaev <klogg> | ||
Submitted: | Fri 21 Sep 2007 03:25:59 PM UTC | ||
Category: | None | Priority: | 3 - Low |
Status: | Done | Privacy: | Public |
Assigned to: | goldsimon | Open/Closed: | Closed |
Planned Release: | None |
Jump to the original submission
Tue 06 Nov 2007 08:54:37 PM UTC, comment #12: |
Simon Goldschmidt <goldsimon> |
Tue 06 Nov 2007 07:32:44 PM UTC, comment #11: Is this really the only writable field that actually does something except setting back a counter? |
Simon Goldschmidt <goldsimon> |
Tue 06 Nov 2007 07:22:52 PM UTC, comment #10:
|
Frédéric Bernon <fbernon> |
Tue 06 Nov 2007 07:18:40 PM UTC, comment #9:
|
Simon Goldschmidt <goldsimon> |
Tue 06 Nov 2007 07:07:58 PM UTC, comment #8: I'm agree, but a little worry to enable to down the netif. Of course, it's more a generic security problem with SNMP, but... But isn't it possible to have a option for that (I would like to be able to use SNMP for statistics and monitoring, but not for management) ?
|
Frédéric Bernon <fbernon> |
Tue 06 Nov 2007 07:02:36 PM UTC, comment #7:
|
Simon Goldschmidt <goldsimon> |
Tue 06 Nov 2007 06:59:13 PM UTC, comment #6: Yep, you're right there.
|
Simon Goldschmidt <goldsimon> |
Fri 02 Nov 2007 02:26:35 PM UTC, comment #5: Shouldn't the #if LWIP_NETIF_LINK_CALLBACK only be around the link specific test?
|
Jonathan Larmour <jifl> |
Fri 02 Nov 2007 02:19:27 PM UTC, comment #4: Attaching a patch including my interpretation of ifOperStatus |
Simon Goldschmidt <goldsimon> |
Fri 02 Nov 2007 02:15:11 PM UTC, comment #3: Adding a patch that is better to read (3 lines before and after each changed line, to get the right context) and the full changed mib2.c file.
|
Simon Goldschmidt <goldsimon> |
Thu 01 Nov 2007 07:58:01 PM UTC, comment #2: I'm not sure to understand your diff file (it's not the format I used). Can you post the whole mib2.c file? Thank you |
Frédéric Bernon <fbernon> |
Thu 01 Nov 2007 07:48:25 PM UTC, comment #1: |
Frédéric Bernon <fbernon> |
Fri 21 Sep 2007 03:25:59 PM UTC, original submission:
ifAdminStatus field in the .interfaces.ifTable.ifEntry can support write operation and actually can bring interface up or down. this patch allows such operation using netif_set_up()/netif_set_down() functions and also add ifOperStatus using netif_is_link_up(). |
Artem Migaev <klogg> |
Depends on the following items: None found
Items that depend on this one: None found
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.
Follow 7 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2007-11-06 | goldsimon | Status | None | Done | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2007-11-02 | goldsimon | Attached File | - | Added ifOperStatus.patch, #14280 | |
2007-11-02 | goldsimon | Attached File | - | Added mib2.c, #14278 | |
Attached File | - | Added mib2.c.patch, #14279 | |||
2007-09-21 | klogg | Attached File | - | Added mib2.diff, #14005 |
Powered by Savane 3.13-758e.
Corresponding source code
I've just checked in this patch. Included a new setting SNMP_SAFE_REQUESTS, which is enabled by default and has to be disabled (defined to 0 in lwipopts.h) to enable ifAdminStatus to be writable!