On Tue, 2008-12-23 at 11:33 -0500, Steve Huston wrote: > Hi Ted, > > Rajith suggested we work out what, if anything, to do for QPID-1449 on > the dev list... I think we were talking about separate patches. You > originally said you had one, but that it shouldn't be necessary > because the base issue would not really be resolved. Yesterday I > believe I resolved the base issue after the same problem showed up in > a different context. I came up with the following patch (it's also > attached to the jira). Tested on Windows ok.
Mea culpa - Speaking as the perpetrator of this bug - I think this fix looks good in general. Clearly the original code is wrong in the case of a 0 length value. I'd check it in to the trunk if I were you. Andrew
