Hi Juan,
On Tue, 2009-06-16 at 08:51 -0700, Juan Lang wrote:
The patch is unfortunately incorrect. Hardcoding 1 as the instance can't work in general. I don't know what the correct fix is offhand. I also don't see how it's a partial revert, as the patch looks nothing like the code before the patch you say it reverts. If you could explain how the call flow is different before and after the reverted patch, that would be a more useful starting point. --Juan
I agree that the patch is not be correct. The revert was for the logic. The original code (which allows MINITAB to work) set the item to 1 when the type was PDU_GETNEXT and the lengths (pVarBind & entryOid) not equal and not off by 1 and the SnmpUtilOidNCmp routine returned a 0. My patch restored that logic.
I will attempt to get a call trace that shows the difference in MINITAB. It will take a few days.
Thanks, Guy