From 60c8ec5f937a28a5ec3d37f5d808c74fc4ef5735 Mon Sep 17 00:00:00 2001 From: Matej Marusak Date: Tue, 5 Sep 2017 10:48:31 +0200 Subject: [PATCH 3/4] client-python: Do not try to unlink None Rhbz: #1416277 https://github.com/abrt/libreport/blob/master/src/client-python/reportclient/dnfdebuginfo.py#L135 returns None when downloading fails. Signed-off-by: Matej Marusak --- src/client-python/reportclient/debuginfo.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/client-python/reportclient/debuginfo.py b/src/client-python/reportclient/debuginfo.py index 0385f716..4390304e 100644 --- a/src/client-python/reportclient/debuginfo.py +++ b/src/client-python/reportclient/debuginfo.py @@ -374,7 +374,8 @@ class DebugInfoDownload(object): # I observed a zero-length file left on error, # which prevents cleanup later. Fix it: try: - os.unlink(package_full_path) + if package_full_path is not None: + os.unlink(package_full_path) except OSError: pass print(_("Downloading package {0} failed").format(pkg)) -- 2.13.5