On Tue, 24 Mar 2009, Florian Köberle wrote:
[...]
> + if (handle_idx < 0 || handle_idx >= schan_handle_count)
> + {
> + ERR("Handle %ld is not in range [0,%ld]\n", handle_idx,
> schan_handle_count-1);
> + return NULL;
> + }
A conformance test to show that Windows has this check would be nice.
Otherwise this patch is just for hiding a bug and thus should not be
committed. But really, finding out why this API gets called with an
invalid handle would be much better... (likely harder too, I know)
--
Francois Gouget <[email protected]> http://fgouget.free.fr/
Linux: Because rebooting is for adding new hardware
