ID: 31772
Updated by: [EMAIL PROTECTED]
Reported By: LarryJAdams at comcast dot net
-Status: Open
+Status: Feedback
Bug Type: SNMP related
Operating System: Win32
PHP Version: 5.0.3
New Comment:
Please provide the unified diff somewhere in the web and add the url to
it here.
Previous Comments:
------------------------------------------------------------------------
[2005-02-01 03:17:57] larryjadams at comcast dot net
I have submitted fixes for the following PHP CVS Branches:
PHP 4.3.x
PHP 5.0.x
PHP 5.1.x
The fix also included the snmpv2 feature set. I hope you all enjoy it
as much as I will.
Larry
------------------------------------------------------------------------
[2005-01-31 01:05:46] LarryJAdams at comcast dot net
Description:
------------
The following code in snmp.c from lines: 794 through 831 is not valid.
If both HAVE_AES and SNMP_VALIDATE_ERR are defined, the ifdef's will not
function properly. Please review the code below.
Reproduce code:
---------------
Incorrect code (starting on 810):
"* symbol on purpose, as it's defined to be the same as the former.
*/
|| !strcasecmp(prot, "AES")) {
#else
) {
s->securityPrivProto = usmAES128PrivProtocol;
s->securityPrivProtoLen =
OIDSIZE(usmAES128PrivProtocol);
return (0);"
Correct Code (starting on 810):
"* symbol on purpose, as it's defined to be the same as the former.
*/
|| !strcasecmp(prot, "AES")) {
s->securityPrivProto = usmAES128PrivProtocol;
s->securityPrivProtoLen =
OIDSIZE(usmAES128PrivProtocol);
return (0);
#else
) {
s->securityPrivProto = usmAES128PrivProtocol;
s->securityPrivProtoLen =
OIDSIZE(usmAES128PrivProtocol);
return (0);"
Expected result:
----------------
N/A. Please review code snippet.
Actual result:
--------------
N/A. Please review code snippet.
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=31772&edit=1
