Re: secur32: check for variable before freeing it

[Henri Verbeet]

2009/10/14 Austin English <austinenglish(a)gmail.com>:
> static void *schan_free_handle(ULONG_PTR handle_idx, enum
> schan_handle_type type)
> {
>    struct schan_handle *handle;
>    void *object;
>
>    if (handle_idx == SCHAN_INVALID_HANDLE) return NULL;
>    if (handle_idx >= schan_handle_count) return NULL;
>
> Seems NULL comes from 'handle_idx >= schan_handle_count'.

How does the attached patch work?