lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 27 Jan 2014 10:22:58 +0000
From:	David Vrabel <>
To:	Annie Li <>
CC:	<>, <>,
	<>, <>,
Subject: Re: [Xen-devel] [PATCH net-next v5] xen-netfront: clean up code in

On 26/01/14 10:12, Annie Li wrote:
> From: Annie Li <>
> This patch removes grant transfer releasing code from netfront, and uses
> gnttab_end_foreign_access to end grant access since
> gnttab_end_foreign_access_ref may fail when the grant entry is
> currently used for reading or writing.
> * clean up grant transfer code kept from old netfront(2.6.18) which grants
> pages for access/map and transfer. But grant transfer is deprecated in current
> netfront, so remove corresponding release code for transfer.
> * release grant access (through gnttab_end_foreign_access) and skb for tx/rx path,
> use get_page to ensure page is released when grant access is completed successfully.
> Xen-blkfront/xen-tpmfront/xen-pcifront also have similar issue, but patches
> for them will be created separately.
> V5: Remove unecessary change in xennet_end_access.
> V4: Revert put_page in gnttab_end_foreign_access, and keep netfront change in
> single patch.
> V3: Changes as suggestion from David Vrabel, ensure pages are not freed untill
> grant acess is ended.
> V2: Improve patch comments.
> Signed-off-by: Annie Li <>

Reviewed-by: David Vrabel <>

I think this should be applied to net (and tagged as a stable candidate)
rather than net-next as this fixes are very big resource leak.

To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to
More majordomo info at

Powered by blists - more mailing lists