|
 |
9a6c41 |
833013 - The port number specified with clientaddr is not used for trap udp socket
|
|
|
9a6c41 |
|
|
|
9a6c41 |
commit 521b4e28b4c794a9d6d929858478d13875246ce3
|
|
|
9a6c41 |
Author: Ivosh <ivosh@users.sourceforge.net>
|
|
|
9a6c41 |
Date: Mon Jul 30 10:53:22 2012 -0700
|
|
|
9a6c41 |
|
|
|
9a6c41 |
CHANGES: libnetsnmp: PATCH 3404876: from hardaker: ability to specify local-bound port in addition to address
|
|
|
9a6c41 |
|
|
|
9a6c41 |
Signed-off-by: Wes Hardaker <hardaker@users.sourceforge.net>
|
|
|
9a6c41 |
|
|
|
9a6c41 |
diff -up net-snmp-5.7.2/include/net-snmp/library/default_store.h.port net-snmp-5.7.2/include/net-snmp/library/default_store.h
|
|
|
9a6c41 |
--- net-snmp-5.7.2/include/net-snmp/library/default_store.h.port 2012-10-10 00:28:58.000000000 +0200
|
|
|
9a6c41 |
+++ net-snmp-5.7.2/include/net-snmp/library/default_store.h 2012-11-13 10:51:34.528804796 +0100
|
|
|
9a6c41 |
@@ -94,6 +94,7 @@ extern "C" {
|
|
|
9a6c41 |
#define NETSNMP_DS_LIB_TSM_USE_PREFIX 39 /* TSM's simple security name mapping */
|
|
|
9a6c41 |
#define NETSNMP_DS_LIB_DONT_LOAD_HOST_FILES 40 /* don't read host.conf files */
|
|
|
9a6c41 |
#define NETSNMP_DS_LIB_DNSSEC_WARN_ONLY 41 /* tread DNSSEC errors as warnings */
|
|
|
9a6c41 |
+#define NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT 42 /* NETSNMP_DS_LIB_CLIENT_ADDR includes address and also port */
|
|
|
9a6c41 |
#define NETSNMP_DS_LIB_MAX_BOOL_ID 48 /* match NETSNMP_DS_MAX_SUBIDS */
|
|
|
9a6c41 |
|
|
|
9a6c41 |
/*
|
|
|
9a6c41 |
diff -up net-snmp-5.7.2/perl/default_store/default_store.pm.port net-snmp-5.7.2/perl/default_store/default_store.pm
|
|
|
9a6c41 |
--- net-snmp-5.7.2/perl/default_store/default_store.pm.port 2012-10-10 00:28:58.000000000 +0200
|
|
|
9a6c41 |
+++ net-snmp-5.7.2/perl/default_store/default_store.pm 2012-11-13 10:51:34.679804190 +0100
|
|
|
9a6c41 |
@@ -63,6 +63,7 @@ use vars qw(@ISA %EXPORT_TAGS @EXPORT_OK
|
|
|
9a6c41 |
NETSNMP_DS_LIB_DISABLE_PERSISTENT_LOAD
|
|
|
9a6c41 |
NETSNMP_DS_LIB_DISABLE_PERSISTENT_SAVE
|
|
|
9a6c41 |
NETSNMP_DS_LIB_APPEND_LOGFILES
|
|
|
9a6c41 |
+ NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT
|
|
|
9a6c41 |
NETSNMP_DS_LIB_MIB_WARNINGS
|
|
|
9a6c41 |
NETSNMP_DS_LIB_SECLEVEL
|
|
|
9a6c41 |
NETSNMP_DS_LIB_SNMPVERSION
|
|
|
9a6c41 |
@@ -161,6 +162,7 @@ use vars qw(@ISA %EXPORT_TAGS @EXPORT_OK
|
|
|
9a6c41 |
NETSNMP_DS_LIB_DISABLE_PERSISTENT_LOAD
|
|
|
9a6c41 |
NETSNMP_DS_LIB_DISABLE_PERSISTENT_SAVE
|
|
|
9a6c41 |
NETSNMP_DS_LIB_APPEND_LOGFILES
|
|
|
9a6c41 |
+ NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT
|
|
|
9a6c41 |
NETSNMP_DS_LIB_MIB_WARNINGS
|
|
|
9a6c41 |
NETSNMP_DS_LIB_SECLEVEL
|
|
|
9a6c41 |
NETSNMP_DS_LIB_SNMPVERSION
|
|
|
9a6c41 |
@@ -299,6 +301,7 @@ None by default.
|
|
|
9a6c41 |
NETSNMP_DS_LIB_DISABLE_PERSISTENT_LOAD
|
|
|
9a6c41 |
NETSNMP_DS_LIB_DISABLE_PERSISTENT_SAVE
|
|
|
9a6c41 |
NETSNMP_DS_LIB_APPEND_LOGFILES
|
|
|
9a6c41 |
+ NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT
|
|
|
9a6c41 |
NETSNMP_DS_LIB_MIB_WARNINGS
|
|
|
9a6c41 |
NETSNMP_DS_LIB_SECLEVEL
|
|
|
9a6c41 |
NETSNMP_DS_LIB_SNMPVERSION
|
|
|
9a6c41 |
diff -up net-snmp-5.7.2/perl/default_store/default_store.xs.port net-snmp-5.7.2/perl/default_store/default_store.xs
|
|
|
9a6c41 |
--- net-snmp-5.7.2/perl/default_store/default_store.xs.port 2012-10-10 00:28:58.000000000 +0200
|
|
|
9a6c41 |
+++ net-snmp-5.7.2/perl/default_store/default_store.xs 2012-11-13 10:51:35.154802043 +0100
|
|
|
9a6c41 |
@@ -1238,6 +1238,17 @@ __END__
|
|
|
9a6c41 |
#endif
|
|
|
9a6c41 |
}
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
+ case 36:
|
|
|
9a6c41 |
+ if (memEQ(name, "NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT", 36)) {
|
|
|
9a6c41 |
+#ifdef NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT
|
|
|
9a6c41 |
+ *iv_return = NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT;
|
|
|
9a6c41 |
+ return PERL_constant_ISIV;
|
|
|
9a6c41 |
+#else
|
|
|
9a6c41 |
+ return PERL_constant_NOTDEF;
|
|
|
9a6c41 |
+#endif
|
|
|
9a6c41 |
+ }
|
|
|
9a6c41 |
+ break;
|
|
|
9a6c41 |
+
|
|
|
9a6c41 |
case 38:
|
|
|
9a6c41 |
return constant_38 (aTHX_ name, iv_return);
|
|
|
9a6c41 |
break;
|
|
|
9a6c41 |
@@ -1245,8 +1256,6 @@ __END__
|
|
|
9a6c41 |
return PERL_constant_NOTFOUND;
|
|
|
9a6c41 |
}
|
|
|
9a6c41 |
|
|
|
9a6c41 |
-
|
|
|
9a6c41 |
-
|
|
|
9a6c41 |
/* autogenerated by "gen" from const-xs.inc */
|
|
|
9a6c41 |
|
|
|
9a6c41 |
MODULE = NetSNMP::default_store PACKAGE = NetSNMP::default_store
|
|
|
9a6c41 |
diff -up net-snmp-5.7.2/perl/default_store/test.pl.port net-snmp-5.7.2/perl/default_store/test.pl
|
|
|
9a6c41 |
--- net-snmp-5.7.2/perl/default_store/test.pl.port 2012-10-10 00:28:58.000000000 +0200
|
|
|
9a6c41 |
+++ net-snmp-5.7.2/perl/default_store/test.pl 2012-11-13 10:51:35.185801918 +0100
|
|
|
9a6c41 |
@@ -52,6 +52,7 @@ BEGIN { $| = 1;
|
|
|
9a6c41 |
"NETSNMP_DS_LIB_DISABLE_PERSISTENT_LOAD" => 35,
|
|
|
9a6c41 |
"NETSNMP_DS_LIB_DISABLE_PERSISTENT_SAVE" => 36,
|
|
|
9a6c41 |
"NETSNMP_DS_LIB_APPEND_LOGFILES" => 37,
|
|
|
9a6c41 |
+ "NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT" => 42,
|
|
|
9a6c41 |
"NETSNMP_DS_LIB_MIB_WARNINGS" => 0,
|
|
|
9a6c41 |
"NETSNMP_DS_LIB_SECLEVEL" => 1,
|
|
|
9a6c41 |
"NETSNMP_DS_LIB_SNMPVERSION" => 2,
|
|
|
9a6c41 |
diff -up net-snmp-5.7.2/snmplib/snmp_api.c.port net-snmp-5.7.2/snmplib/snmp_api.c
|
|
|
9a6c41 |
--- net-snmp-5.7.2/snmplib/snmp_api.c.port 2012-10-10 00:28:58.000000000 +0200
|
|
|
9a6c41 |
+++ net-snmp-5.7.2/snmplib/snmp_api.c 2012-11-13 10:51:35.188801906 +0100
|
|
|
9a6c41 |
@@ -769,6 +769,8 @@ register_default_handlers(void)
|
|
|
9a6c41 |
NETSNMP_DS_LIBRARY_ID, NETSNMP_DS_LIB_16BIT_IDS);
|
|
|
9a6c41 |
netsnmp_ds_register_premib(ASN_OCTET_STR, "snmp", "clientaddr",
|
|
|
9a6c41 |
NETSNMP_DS_LIBRARY_ID, NETSNMP_DS_LIB_CLIENT_ADDR);
|
|
|
9a6c41 |
+ netsnmp_ds_register_premib(ASN_BOOLEAN, "snmp", "clientaddrUsesPort",
|
|
|
9a6c41 |
+ NETSNMP_DS_LIBRARY_ID, NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT);
|
|
|
9a6c41 |
netsnmp_ds_register_config(ASN_INTEGER, "snmp", "serverSendBuf",
|
|
|
9a6c41 |
NETSNMP_DS_LIBRARY_ID, NETSNMP_DS_LIB_SERVERSENDBUF);
|
|
|
9a6c41 |
netsnmp_ds_register_config(ASN_INTEGER, "snmp", "serverRecvBuf",
|
|
|
9a6c41 |
diff -up net-snmp-5.7.2/snmplib/transports/snmpUDPIPv4BaseDomain.c.port net-snmp-5.7.2/snmplib/transports/snmpUDPIPv4BaseDomain.c
|
|
|
9a6c41 |
--- net-snmp-5.7.2/snmplib/transports/snmpUDPIPv4BaseDomain.c.port 2012-11-13 10:51:32.781812675 +0100
|
|
|
9a6c41 |
+++ net-snmp-5.7.2/snmplib/transports/snmpUDPIPv4BaseDomain.c 2012-11-13 10:51:35.190801898 +0100
|
|
|
9a6c41 |
@@ -183,8 +183,29 @@ netsnmp_udpipv4base_transport(struct soc
|
|
|
9a6c41 |
NETSNMP_DS_LIB_CLIENT_ADDR);
|
|
|
9a6c41 |
if (client_socket) {
|
|
|
9a6c41 |
struct sockaddr_in client_addr;
|
|
|
9a6c41 |
- netsnmp_sockaddr_in2(&client_addr, client_socket, NULL);
|
|
|
9a6c41 |
- client_addr.sin_port = 0;
|
|
|
9a6c41 |
+
|
|
|
9a6c41 |
+ char *client_address = client_socket;
|
|
|
9a6c41 |
+ int uses_port = netsnmp_ds_get_boolean(NETSNMP_DS_LIBRARY_ID,
|
|
|
9a6c41 |
+ NETSNMP_DS_LIB_CLIENT_ADDR_USES_PORT);
|
|
|
9a6c41 |
+ if ((uses_port == 1) && (strchr(client_socket, ':') == NULL)) {
|
|
|
9a6c41 |
+ client_address = malloc(strlen(client_socket) + 3);
|
|
|
9a6c41 |
+ if (client_address == NULL) {
|
|
|
9a6c41 |
+ netsnmp_socketbase_close(t);
|
|
|
9a6c41 |
+ netsnmp_transport_free(t);
|
|
|
9a6c41 |
+ return NULL;
|
|
|
9a6c41 |
+ } /* if NETSNMP_DS_LIB_CLIENT_ADDR */
|
|
|
9a6c41 |
+ strcpy(client_address, client_socket); /* expects a port but there is none */
|
|
|
9a6c41 |
+ strcat(client_address, ":0"); /* specified then provide ephemeral one */
|
|
|
9a6c41 |
+ }
|
|
|
9a6c41 |
+
|
|
|
9a6c41 |
+ netsnmp_sockaddr_in2(&client_addr, client_address, NULL);
|
|
|
9a6c41 |
+ if (uses_port == 0) {
|
|
|
9a6c41 |
+ client_addr.sin_port = 0;
|
|
|
9a6c41 |
+ }
|
|
|
9a6c41 |
+ if (client_address != client_socket) {
|
|
|
9a6c41 |
+ free(client_address);
|
|
|
9a6c41 |
+ }
|
|
|
9a6c41 |
+
|
|
|
9a6c41 |
DEBUGMSGTL(("netsnmp_udpbase", "binding socket: %d\n", t->sock));
|
|
|
9a6c41 |
rc = bind(t->sock, (struct sockaddr *)&client_addr,
|
|
|
9a6c41 |
sizeof(struct sockaddr));
|
|
|
9a6c41 |
commit 9e00fff692081e36c9d883fab7b6bd8881c670fc
|
|
|
9a6c41 |
Author: Jan Safranek <jsafranek@users.sourceforge.net>
|
|
|
9a6c41 |
Date: Tue Aug 7 12:27:18 2012 +0200
|
|
|
9a6c41 |
|
|
|
9a6c41 |
Document new clientaddrUsesPort option.
|
|
|
9a6c41 |
|
|
|
9a6c41 |
diff --git a/man/snmp.conf.5.def b/man/snmp.conf.5.def
|
|
|
9a6c41 |
index 9c7c55b..904635b 100644
|
|
|
9a6c41 |
--- a/man/snmp.conf.5.def
|
|
|
9a6c41 |
+++ b/man/snmp.conf.5.def
|
|
|
9a6c41 |
@@ -127,6 +127,10 @@ This value is also used by \fBsnmpd\fR when generating notifications.
|
|
|
9a6c41 |
.\" But not responses to an incoming request?
|
|
|
9a6c41 |
.\" What about snmptrapd?
|
|
|
9a6c41 |
.\"
|
|
|
9a6c41 |
+.IP "clientaddrUsesPort no"
|
|
|
9a6c41 |
+specifies, if clientaddr option contains a port number. Set this option
|
|
|
9a6c41 |
+to "yes", if clientaddr contains a port number and this port should
|
|
|
9a6c41 |
+be used for sending outgoing SNMP requests.
|
|
|
9a6c41 |
.IP "clientRecvBuf INTEGER"
|
|
|
9a6c41 |
specifies the desired size of the buffer to be used when receiving
|
|
|
9a6c41 |
responses to SNMP requests.
|
|
|
9a6c41 |
|