[Bug 18786] patch

Guy Albertelli galberte at neo.rr.com
Wed Jun 17 19:47:55 CDT 2009


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




More information about the wine-devel mailing list