[ros-dev] [ros-diffs] [zhu] 71644: Implemented IRQ cancelling for listen() and connect() calls.

[Thomas Faber]

On 2016-06-15 12:10, zhu at svn.reactos.org wrote:
> --- branches/GSoC_2016/lwIP/drivers/network/tcpip/main.c	[iso-8859-1] (original)
> +++ branches/GSoC_2016/lwIP/drivers/network/tcpip/main.c	[iso-8859-1] Wed Jun 15 19:10:40 2016
> @@ -384,7 +384,9 @@
>   Quickie:
>       Irp->IoStatus.Status = Status;
>
> -    return Irp->IoStatus.Status;
> +	IoCompleteRequest(Irp, IO_NETWORK_INCREMENT);
> +	
> +	return Irp->IoStatus.Status;
>   }
>
>   static

This is illegal. IoCompleteRequest frees the IRP so you must not access
it afterwards, and use the local variable instead.