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)