dnf/backport-Add-provide-exception-handling.patch
sun_hai_10 8c2b81ae7b sync upstream patch
(cherry picked from commit 2f05a5ccc561ca16a40487c7b4f5d35decd2bf44)
2023-08-14 10:01:44 +08:00

113 lines
3.9 KiB
Diff

From 0a6a991342d9ac798ba4d91c04028161e726ec81 Mon Sep 17 00:00:00 2001
From: zengwei2000 <102871671+zengwei2000@users.noreply.github.com>
Date: Thu, 6 Jul 2023 05:23:18 +0000
Subject: [PATCH] Add provide exception handling
Conflict:NA
Reference:https://github.com/rpm-software-management/dnf/commit/0a6a991342d9ac798ba4d91c04028161e726ec81
---
dnf/cli/utils.py | 90 +++++++++++++++++++++++++-----------------------
1 file changed, 47 insertions(+), 43 deletions(-)
diff --git a/dnf/cli/utils.py b/dnf/cli/utils.py
index 1c3db758a1..f00e33530f 100644
--- a/dnf/cli/utils.py
+++ b/dnf/cli/utils.py
@@ -60,49 +60,53 @@ def seconds_to_ui_time(seconds):
def get_process_info(pid):
"""Return info dict about a process."""
- pid = int(pid)
-
- # Maybe true if /proc isn't mounted, or not Linux ... or something.
- if (not os.path.exists("/proc/%d/status" % pid) or
- not os.path.exists("/proc/stat") or
- not os.path.exists("/proc/%d/stat" % pid)):
- return
-
- ps = {}
- with open("/proc/%d/status" % pid) as status_file:
- for line in status_file:
- if line[-1] != '\n':
- continue
- data = line[:-1].split(':\t', 1)
- if len(data) < 2:
- continue
- data[1] = dnf.util.rtrim(data[1], ' kB')
- ps[data[0].strip().lower()] = data[1].strip()
- if 'vmrss' not in ps:
- return
- if 'vmsize' not in ps:
- return
-
- boot_time = None
- with open("/proc/stat") as stat_file:
- for line in stat_file:
- if line.startswith("btime "):
- boot_time = int(line[len("btime "):-1])
- break
- if boot_time is None:
- return
-
- with open('/proc/%d/stat' % pid) as stat_file:
- ps_stat = stat_file.read().split()
- ps['start_time'] = boot_time + jiffies_to_seconds(ps_stat[21])
- ps['state'] = {'R' : _('Running'),
- 'S' : _('Sleeping'),
- 'D' : _('Uninterruptible'),
- 'Z' : _('Zombie'),
- 'T' : _('Traced/Stopped')
- }.get(ps_stat[2], _('Unknown'))
-
- return ps
+ try:
+ pid = int(pid)
+
+ # Maybe true if /proc isn't mounted, or not Linux ... or something.
+ if (not os.path.exists("/proc/%d/status" % pid) or
+ not os.path.exists("/proc/stat") or
+ not os.path.exists("/proc/%d/stat" % pid)):
+ return None
+
+ ps = {}
+ with open("/proc/%d/status" % pid) as status_file:
+ for line in status_file:
+ if line[-1] != '\n':
+ continue
+ data = line[:-1].split(':\t', 1)
+ if len(data) < 2:
+ continue
+ data[1] = dnf.util.rtrim(data[1], ' kB')
+ ps[data[0].strip().lower()] = data[1].strip()
+ if 'vmrss' not in ps:
+ return None
+ if 'vmsize' not in ps:
+ return None
+
+ boot_time = None
+ with open("/proc/stat") as stat_file:
+ for line in stat_file:
+ if line.startswith("btime "):
+ boot_time = int(line[len("btime "):-1])
+ break
+ if boot_time is None:
+ return None
+
+ with open('/proc/%d/stat' % pid) as stat_file:
+ ps_stat = stat_file.read().split()
+ ps['start_time'] = boot_time + jiffies_to_seconds(ps_stat[21])
+ ps['state'] = {'R' : _('Running'),
+ 'S' : _('Sleeping'),
+ 'D' : _('Uninterruptible'),
+ 'Z' : _('Zombie'),
+ 'T' : _('Traced/Stopped')
+ }.get(ps_stat[2], _('Unknown'))
+
+ return ps
+ except (OSError, ValueError) as e:
+ logger.error("Failed to get process info: %s", e)
+ return None
def show_lock_owner(pid):