Blame SOURCES/net-snmp-5.7.2-extend-reload.patch

34d7f3
1228893 - snmpd segfaults on reload with multiple 'exec' entry
34d7f3
34d7f3
commit 54290bede338164ca65a3eed224fa5040d7dd857
34d7f3
Author: Niels Baggesen <nba@users.sourceforge.net>
34d7f3
Date:   Fri Oct 26 07:50:11 2012 +0200
34d7f3
34d7f3
    Better cleanup of "exec" config to avoid crash during reload of config
34d7f3
34d7f3
diff --git a/agent/mibgroup/agent/extend.c b/agent/mibgroup/agent/extend.c
34d7f3
index 68a11e2..9cb641c 100644
34d7f3
--- a/agent/mibgroup/agent/extend.c
34d7f3
+++ b/agent/mibgroup/agent/extend.c
34d7f3
@@ -34,7 +34,7 @@ typedef struct extend_registration_block_s {
34d7f3
     size_t              oid_len;
34d7f3
     long                num_entries;
34d7f3
     netsnmp_extend     *ehead;
34d7f3
-    netsnmp_handler_registration       *reg[3];
34d7f3
+    netsnmp_handler_registration       *reg[4];
34d7f3
     struct extend_registration_block_s *next;
34d7f3
 } extend_registration_block;
34d7f3
 extend_registration_block *ereg_head = NULL;
34d7f3
@@ -222,10 +222,13 @@ _register_extend( oid *base, size_t len )
34d7f3
     rc = netsnmp_register_watched_scalar2( reg, winfo );
34d7f3
     if (rc != SNMPERR_SUCCESS)
34d7f3
         goto bail;
34d7f3
+    eptr->reg[3] = reg;
34d7f3
 
34d7f3
     return eptr;
34d7f3
 
34d7f3
 bail:
34d7f3
+    if (eptr->reg[3])
34d7f3
+        netsnmp_unregister_handler(eptr->reg[3]);
34d7f3
     if (eptr->reg[2])
34d7f3
         netsnmp_unregister_handler(eptr->reg[2]);
34d7f3
     if (eptr->reg[1])
34d7f3
@@ -267,6 +270,7 @@ extend_clear_callback(int majorID, int minorID,
34d7f3
         netsnmp_unregister_handler( eptr->reg[0] );
34d7f3
         netsnmp_unregister_handler( eptr->reg[1] );
34d7f3
         netsnmp_unregister_handler( eptr->reg[2] );
34d7f3
+        netsnmp_unregister_handler( eptr->reg[3] );
34d7f3
         SNMP_FREE(eptr);
34d7f3
     }
34d7f3
     ereg_head = NULL;
34d7f3
@@ -550,6 +554,10 @@ extend_parse_config(const char *token, char *cptr)
34d7f3
     }
34d7f3
 
34d7f3
     eptr      = _register_extend( oid_buf, oid_len );
34d7f3
+    if (!eptr) {
34d7f3
+        snmp_log(LOG_ERR, "Failed to register extend entry '%s' - possibly duplicate name.\n", exec_name );
34d7f3
+        return;
34d7f3
+    }
34d7f3
     extension = _new_extension( exec_name, flags, eptr );
34d7f3
     if (extension) {
34d7f3
         extension->command  = strdup( exec_command );