[pacman-dev] [PATCH] Reset dload_interrupted early in curl_download_internal()
Lukas Fleischer
archlinux at cryptocrack.de
Thu Aug 18 03:13:31 EDT 2011
On Wed, Aug 17, 2011 at 04:54:56PM +0200, Lukas Fleischer wrote:
> Avoid invoking unlink() with a NULL path if the URL isn't to a file and
> we fail to create a temporary file.
>
> Signed-off-by: Lukas Fleischer <archlinux at cryptocrack.de>
> ---
> lib/libalpm/dload.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
Note that this still isn't fixed with the latest refactoring (not sure
if 9f139550 was supposed to fix this). If we fail to create a temporary
file (around line 224), tempfile still points to NULL and the unlink()
invocation might result in a segfault.
More information about the pacman-dev
mailing list