fixed typos in tests

Boris Lytochkin · Boris Lytochkin · commit dbed284039fe · 2011-05-10T20:04:26.000Z
ucd-snmp epoch is over, drop legacy memory allocation (non e*)
tune 'buggy snmp_snprint_value' detection on some systems
diff --git a/ext/snmp/config.m4 b/ext/snmp/config.m4
@@ -62,7 +62,7 @@ if test "$PHP_SNMP" != "no"; then
   dnl Check for buggy snmp_snprint_value() (net-snmp BUGid 2027834)
   AC_CACHE_CHECK([for buggy snmp_snprint_value], ac_cv_buggy_snprint_value,[
     save_CFLAGS="$CFLAGS"
-    CFLAGS="$CFLAGS -I${SNMP_PREFIX}/include"
+    CFLAGS="$CFLAGS -I${SNMP_PREFIX}/include $SNMP_SHARED_LIBADD"
     AC_TRY_RUN( [
 #include <stdio.h>
 #include <stdlib.h>
diff --git a/ext/snmp/snmp.c b/ext/snmp/snmp.c
@@ -440,12 +440,20 @@ static PHP_GINIT_FUNCTION(snmp)
 }
 /* }}} */
 
+#define PHP_SNMP_SESSION_FREE(a) { \
+	if ((*session)->a) { \
+		efree((*session)->a); \
+		(*session)->a = NULL; \
+	} \
+}
+
 static void netsnmp_session_free(php_snmp_session **session)
 {
 	if (*session) {
-		if ((*session)->peername) {
-			efree((*session)->peername);
-		}
+		PHP_SNMP_SESSION_FREE(peername);
+		PHP_SNMP_SESSION_FREE(community);
+		PHP_SNMP_SESSION_FREE(securityName);
+		PHP_SNMP_SESSION_FREE(contextEngineID);
 		efree(*session);
 		*session = NULL;
 	}
@@ -1115,11 +1123,11 @@ static int netsnmp_session_init(php_snmp_session **session_p, int version, char
 
 	if (version == SNMP_VERSION_3) {
 		/* Setting the security name. */
-		session->securityName = strdup(community);
+		session->securityName = estrdup(community);
 		session->securityNameLen = strlen(session->securityName);
 	} else {
 		session->authenticator = NULL;
-		session->community = (u_char *)community;
+		session->community = (u_char *)estrdup(community);
 		session->community_len = strlen(community);
 	}
 
@@ -1222,20 +1230,20 @@ static int netsnmp_session_gen_sec_key(struct snmp_session *s, char *pass TSRMLS
 static int netsnmp_session_set_contextEngineID(struct snmp_session *s, char * contextEngineID TSRMLS_DC)
 {
 	size_t	ebuf_len = 32, eout_len = 0;
-	u_char	*ebuf = (u_char *) malloc(ebuf_len); /* memory freed by SNMP library, malloc NOT emalloc */
+	u_char	*ebuf = (u_char *) emalloc(ebuf_len);
 
 	if (ebuf == NULL) {
 		php_error_docref(NULL TSRMLS_CC, E_WARNING, "malloc failure setting contextEngineID");
 		return (-1);
 	}
 	if (!snmp_hex_to_binary(&ebuf, &ebuf_len, &eout_len, 1, contextEngineID)) {
 		php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bad engine ID value '%s'", contextEngineID);
-		free(ebuf);
+		efree(ebuf);
 		return (-1);
 	}
 
 	if (s->contextEngineID) {
-		free(s->contextEngineID);
+		efree(s->contextEngineID);
 	}
 
 	s->contextEngineID = ebuf;
diff --git a/ext/snmp/tests/snmp-object-errno-errstr.phpt b/ext/snmp/tests/snmp-object-errno-errstr.phpt
@@ -1,5 +1,5 @@
 --TEST--                                 
-OO API: getErrno & getError functions
+OO API: getErrno & getError methods
 --CREDITS--
 Boris Lytochkin
 --SKIPIF--
diff --git a/ext/snmp/tests/wrong_hostname.phpt b/ext/snmp/tests/wrong_hostname.phpt
@@ -1,5 +1,5 @@
 --TEST--                                 
-Generic timeout (wrong community)
+Wrong hostname
 --CREDITS--
 Boris Lytochkin
 --SKIPIF--
