[PATCH 3/4] ntdll: Validate job handles at process creation.

Alexandre Julliard julliard at winehq.org
Thu May 20 02:56:36 CDT 2021


Paul Gofman <pgofman at codeweavers.com> writes:

> +    job_handle_count = req->jobs_size / sizeof(*handles);
> +    for (i = 0; i < job_handle_count; ++i)
> +    {
> +        if (!(job = get_job_obj( current->process, job_handles[i], JOB_OBJECT_ASSIGN_PROCESS )))
> +        {
> +            close( socket_fd );
> +            goto done;
> +        }
> +        release_object( job );
> +    }

It doesn't seem useful to pre-check the handles, you'll need to handle
failures when assigning the process anyway.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list