Compare commits
No commits in common. 'c9' and 'c8' have entirely different histories.
@ -1 +1 @@
|
|||||||
SOURCES/psutil-5.8.0.tar.gz
|
SOURCES/psutil-5.4.3.tar.gz
|
||||||
|
@ -1 +1 @@
|
|||||||
4561d6c7747bdcc6764bc14510415016016522b0 SOURCES/psutil-5.8.0.tar.gz
|
d1335bf6850e57fe6fa758caf74fced68ca66d92 SOURCES/psutil-5.4.3.tar.gz
|
||||||
|
@ -0,0 +1,600 @@
|
|||||||
|
diff --git a/psutil/_psutil_aix.c b/psutil/_psutil_aix.c
|
||||||
|
index 916254d..37a1fb2 100644
|
||||||
|
--- a/psutil/_psutil_aix.c
|
||||||
|
+++ b/psutil/_psutil_aix.c
|
||||||
|
@@ -158,8 +158,8 @@ psutil_proc_name_and_args(PyObject *self, PyObject *args) {
|
||||||
|
py_retlist = Py_BuildValue("OO", py_name, py_args);
|
||||||
|
if (!py_retlist)
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_name);
|
||||||
|
- Py_DECREF(py_args);
|
||||||
|
+ Py_CLEAR(py_name);
|
||||||
|
+ Py_CLEAR(py_args);
|
||||||
|
return py_retlist;
|
||||||
|
|
||||||
|
error:
|
||||||
|
@@ -389,10 +389,10 @@ psutil_users(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_username);
|
||||||
|
- Py_DECREF(py_tty);
|
||||||
|
- Py_DECREF(py_hostname);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_username);
|
||||||
|
+ Py_CLEAR(py_tty);
|
||||||
|
+ Py_CLEAR(py_hostname);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
endutxent();
|
||||||
|
|
||||||
|
@@ -449,9 +449,9 @@ psutil_disk_partitions(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_dev);
|
||||||
|
- Py_DECREF(py_mountp);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_dev);
|
||||||
|
+ Py_CLEAR(py_mountp);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
mt = getmntent(file);
|
||||||
|
}
|
||||||
|
endmntent(file);
|
||||||
|
diff --git a/psutil/_psutil_bsd.c b/psutil/_psutil_bsd.c
|
||||||
|
index 9a2ed04..3dc04e0 100644
|
||||||
|
--- a/psutil/_psutil_bsd.c
|
||||||
|
+++ b/psutil/_psutil_bsd.c
|
||||||
|
@@ -152,7 +152,7 @@ psutil_pids(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_pid))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_pid);
|
||||||
|
+ Py_CLEAR(py_pid);
|
||||||
|
proclist++;
|
||||||
|
}
|
||||||
|
free(orig_address);
|
||||||
|
@@ -507,8 +507,8 @@ psutil_proc_open_files(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_path);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_path);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
free(freep);
|
||||||
|
@@ -670,9 +670,9 @@ psutil_disk_partitions(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_dev);
|
||||||
|
- Py_DECREF(py_mountp);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_dev);
|
||||||
|
+ Py_CLEAR(py_mountp);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
free(fs);
|
||||||
|
@@ -765,7 +765,7 @@ psutil_net_io_counters(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItemString(py_retdict, ifc_name, py_ifc_info))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_ifc_info);
|
||||||
|
+ Py_CLEAR(py_ifc_info);
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
continue;
|
||||||
|
@@ -840,10 +840,10 @@ psutil_users(PyObject *self, PyObject *args) {
|
||||||
|
fclose(fp);
|
||||||
|
goto error;
|
||||||
|
}
|
||||||
|
- Py_DECREF(py_username);
|
||||||
|
- Py_DECREF(py_tty);
|
||||||
|
- Py_DECREF(py_hostname);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_username);
|
||||||
|
+ Py_CLEAR(py_tty);
|
||||||
|
+ Py_CLEAR(py_hostname);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
fclose(fp);
|
||||||
|
@@ -883,10 +883,10 @@ psutil_users(PyObject *self, PyObject *args) {
|
||||||
|
endutxent();
|
||||||
|
goto error;
|
||||||
|
}
|
||||||
|
- Py_DECREF(py_username);
|
||||||
|
- Py_DECREF(py_tty);
|
||||||
|
- Py_DECREF(py_hostname);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_username);
|
||||||
|
+ Py_CLEAR(py_tty);
|
||||||
|
+ Py_CLEAR(py_hostname);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
endutxent();
|
||||||
|
diff --git a/psutil/_psutil_linux.c b/psutil/_psutil_linux.c
|
||||||
|
index d1f0d14..00212ba 100644
|
||||||
|
--- a/psutil/_psutil_linux.c
|
||||||
|
+++ b/psutil/_psutil_linux.c
|
||||||
|
@@ -232,9 +232,9 @@ psutil_disk_partitions(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_dev);
|
||||||
|
- Py_DECREF(py_mountp);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_dev);
|
||||||
|
+ Py_CLEAR(py_mountp);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
endmntent(file);
|
||||||
|
return py_retlist;
|
||||||
|
@@ -488,10 +488,10 @@ psutil_users(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_username);
|
||||||
|
- Py_DECREF(py_tty);
|
||||||
|
- Py_DECREF(py_hostname);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_username);
|
||||||
|
+ Py_CLEAR(py_tty);
|
||||||
|
+ Py_CLEAR(py_hostname);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
endutent();
|
||||||
|
return py_retlist;
|
||||||
|
diff --git a/psutil/_psutil_osx.c b/psutil/_psutil_osx.c
|
||||||
|
index 55dd64c..37f5284 100644
|
||||||
|
--- a/psutil/_psutil_osx.c
|
||||||
|
+++ b/psutil/_psutil_osx.c
|
||||||
|
@@ -795,7 +795,7 @@ psutil_per_cpu_times(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_cputime))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_cputime);
|
||||||
|
+ Py_CLEAR(py_cputime);
|
||||||
|
}
|
||||||
|
|
||||||
|
ret = vm_deallocate(mach_task_self(), (vm_address_t)info_array,
|
||||||
|
@@ -977,9 +977,9 @@ psutil_disk_partitions(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_dev);
|
||||||
|
- Py_DECREF(py_mountp);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_dev);
|
||||||
|
+ Py_CLEAR(py_mountp);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
free(fs);
|
||||||
|
@@ -1055,7 +1055,6 @@ psutil_proc_threads(PyObject *self, PyObject *args) {
|
||||||
|
}
|
||||||
|
|
||||||
|
for (j = 0; j < thread_count; j++) {
|
||||||
|
- py_tuple = NULL;
|
||||||
|
thread_info_count = THREAD_INFO_MAX;
|
||||||
|
kr = thread_info(thread_list[j], THREAD_BASIC_INFO,
|
||||||
|
(thread_info_t)thinfo_basic, &thread_info_count);
|
||||||
|
@@ -1076,7 +1075,7 @@ psutil_proc_threads(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
ret = vm_deallocate(task, (vm_address_t)thread_list,
|
||||||
|
@@ -1185,10 +1184,8 @@ psutil_proc_open_files(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
- py_tuple = NULL;
|
||||||
|
- Py_DECREF(py_path);
|
||||||
|
- py_path = NULL;
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
+ Py_CLEAR(py_path);
|
||||||
|
// --- /construct python list
|
||||||
|
}
|
||||||
|
}
|
||||||
|
@@ -1368,7 +1365,7 @@ psutil_proc_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
else if (family == AF_UNIX) {
|
||||||
|
py_laddr = PyUnicode_DecodeFSDefault(
|
||||||
|
@@ -1390,9 +1387,9 @@ psutil_proc_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
- Py_DECREF(py_laddr);
|
||||||
|
- Py_DECREF(py_raddr);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
+ Py_CLEAR(py_laddr);
|
||||||
|
+ Py_CLEAR(py_raddr);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
@@ -1513,7 +1510,7 @@ psutil_net_io_counters(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItemString(py_retdict, ifc_name, py_ifc_info))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_ifc_info);
|
||||||
|
+ Py_CLEAR(py_ifc_info);
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
continue;
|
||||||
|
@@ -1686,7 +1683,7 @@ psutil_disk_io_counters(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItemString(py_retdict, disk_name, py_disk_info))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_disk_info);
|
||||||
|
+ Py_CLEAR(py_disk_info);
|
||||||
|
|
||||||
|
CFRelease(parent_dict);
|
||||||
|
IOObjectRelease(parent);
|
||||||
|
@@ -1748,10 +1745,10 @@ psutil_users(PyObject *self, PyObject *args) {
|
||||||
|
endutxent();
|
||||||
|
goto error;
|
||||||
|
}
|
||||||
|
- Py_DECREF(py_username);
|
||||||
|
- Py_DECREF(py_tty);
|
||||||
|
- Py_DECREF(py_hostname);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_username);
|
||||||
|
+ Py_CLEAR(py_tty);
|
||||||
|
+ Py_CLEAR(py_hostname);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
endutxent();
|
||||||
|
diff --git a/psutil/_psutil_posix.c b/psutil/_psutil_posix.c
|
||||||
|
index cc82727..6d9212a 100644
|
||||||
|
--- a/psutil/_psutil_posix.c
|
||||||
|
+++ b/psutil/_psutil_posix.c
|
||||||
|
@@ -324,11 +324,11 @@ psutil_net_if_addrs(PyObject* self, PyObject* args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
- Py_DECREF(py_address);
|
||||||
|
- Py_DECREF(py_netmask);
|
||||||
|
- Py_DECREF(py_broadcast);
|
||||||
|
- Py_DECREF(py_ptp);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
+ Py_CLEAR(py_address);
|
||||||
|
+ Py_CLEAR(py_netmask);
|
||||||
|
+ Py_CLEAR(py_broadcast);
|
||||||
|
+ Py_CLEAR(py_ptp);
|
||||||
|
}
|
||||||
|
|
||||||
|
freeifaddrs(ifaddr);
|
||||||
|
diff --git a/psutil/_psutil_sunos.c b/psutil/_psutil_sunos.c
|
||||||
|
index c667364..ff635f6 100644
|
||||||
|
--- a/psutil/_psutil_sunos.c
|
||||||
|
+++ b/psutil/_psutil_sunos.c
|
||||||
|
@@ -214,8 +214,8 @@ psutil_proc_environ(PyObject *self, PyObject *args) {
|
||||||
|
if (PyDict_SetItem(py_retdict, py_envname, py_envval) < 0)
|
||||||
|
goto error;
|
||||||
|
|
||||||
|
- Py_DECREF(py_envname);
|
||||||
|
- Py_DECREF(py_envval);
|
||||||
|
+ Py_CLEAR(py_envname);
|
||||||
|
+ Py_CLEAR(py_envval);
|
||||||
|
}
|
||||||
|
|
||||||
|
psutil_free_cstrings_array(env, env_count);
|
||||||
|
@@ -576,10 +576,10 @@ psutil_users(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_username);
|
||||||
|
- Py_DECREF(py_tty);
|
||||||
|
- Py_DECREF(py_hostname);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_username);
|
||||||
|
+ Py_CLEAR(py_tty);
|
||||||
|
+ Py_CLEAR(py_hostname);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
endutxent();
|
||||||
|
|
||||||
|
@@ -635,9 +635,9 @@ psutil_disk_partitions(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_dev);
|
||||||
|
- Py_DECREF(py_mountp);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_dev);
|
||||||
|
+ Py_CLEAR(py_mountp);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
fclose(file);
|
||||||
|
return py_retlist;
|
||||||
|
@@ -688,8 +688,7 @@ psutil_per_cpu_times(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_cputime))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_cputime);
|
||||||
|
- py_cputime = NULL;
|
||||||
|
+ Py_CLEAR(py_cputime);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -745,7 +744,7 @@ psutil_disk_io_counters(PyObject *self, PyObject *args) {
|
||||||
|
if (PyDict_SetItemString(py_retdict, ksp->ks_name,
|
||||||
|
py_disk_info))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_disk_info);
|
||||||
|
+ Py_CLEAR(py_disk_info);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
ksp = ksp->ks_next;
|
||||||
|
@@ -880,8 +879,8 @@ psutil_proc_memory_maps(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_path);
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_path);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
|
||||||
|
// increment pointer
|
||||||
|
p += 1;
|
||||||
|
@@ -996,7 +995,7 @@ psutil_net_io_counters(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItemString(py_retdict, ksp->ks_name, py_ifc_info))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_ifc_info);
|
||||||
|
+ Py_CLEAR(py_ifc_info);
|
||||||
|
goto next;
|
||||||
|
|
||||||
|
next:
|
||||||
|
@@ -1189,7 +1188,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
#if defined(AF_INET6)
|
||||||
|
@@ -1232,7 +1231,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
#endif
|
||||||
|
@@ -1240,7 +1239,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
else if (mibhdr->level == MIB2_UDP || mibhdr->level == MIB2_UDP_ENTRY) {
|
||||||
|
ude = (mib2_udpEntry_t *)databuf.buf;
|
||||||
|
num_ent = mibhdr->len / sizeof(mib2_udpEntry_t);
|
||||||
|
- assert(num_ent * sizeof(mib2_udpEntry_t) == mibhdr->len);
|
||||||
|
+ assert(num_ent * sizeof(mib2_udpEntry_t) == mibhdr.len);
|
||||||
|
for (i = 0; i < num_ent; i++, ude++) {
|
||||||
|
#ifdef NEW_MIB_COMPLIANT
|
||||||
|
processed_pid = ude->udpCreationProcess;
|
||||||
|
@@ -1271,7 +1270,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
#if defined(AF_INET6)
|
||||||
|
@@ -1304,7 +1303,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
#endif
|
||||||
|
@@ -1477,7 +1476,7 @@ psutil_net_if_stats(PyObject* self, PyObject* args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItemString(py_retdict, ksp->ks_name, py_ifc_info))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_ifc_info);
|
||||||
|
+ Py_CLEAR(py_ifc_info);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
diff --git a/psutil/_psutil_windows.c b/psutil/_psutil_windows.c
|
||||||
|
index 81d1b4a..adef70c 100644
|
||||||
|
--- a/psutil/_psutil_windows.c
|
||||||
|
+++ b/psutil/_psutil_windows.c
|
||||||
|
@@ -307,7 +307,7 @@ psutil_pids(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_pid))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_pid);
|
||||||
|
+ Py_CLEAR(py_pid);
|
||||||
|
}
|
||||||
|
|
||||||
|
// free C array allocated for PIDs
|
||||||
|
@@ -1019,7 +1019,7 @@ psutil_per_cpu_times(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
free(sppi);
|
||||||
|
@@ -1237,7 +1237,7 @@ psutil_proc_threads(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
|
||||||
|
CloseHandle(hThread);
|
||||||
|
}
|
||||||
|
@@ -1695,7 +1695,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_conn_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_conn_tuple);
|
||||||
|
+ Py_CLEAR(py_conn_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
@@ -1792,7 +1792,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_conn_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_conn_tuple);
|
||||||
|
+ Py_CLEAR(py_conn_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
@@ -1866,7 +1866,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_conn_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_conn_tuple);
|
||||||
|
+ Py_CLEAR(py_conn_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
@@ -1939,7 +1939,7 @@ psutil_net_connections(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_conn_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_conn_tuple);
|
||||||
|
+ Py_CLEAR(py_conn_tuple);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
@@ -2331,8 +2331,8 @@ psutil_net_io_counters(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItem(py_retdict, py_nic_name, py_nic_info))
|
||||||
|
goto error;
|
||||||
|
- Py_XDECREF(py_nic_name);
|
||||||
|
- Py_XDECREF(py_nic_info);
|
||||||
|
+ Py_CLEAR(py_nic_name);
|
||||||
|
+ Py_CLEAR(py_nic_info);
|
||||||
|
|
||||||
|
free(pIfRow);
|
||||||
|
pCurrAddresses = pCurrAddresses->Next;
|
||||||
|
@@ -2446,7 +2446,7 @@ psutil_disk_io_counters(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItemString(py_retdict, szDeviceDisplay, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_XDECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
|
||||||
|
next:
|
||||||
|
CloseHandle(hDevice);
|
||||||
|
@@ -2603,7 +2603,7 @@ psutil_disk_partitions(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
}
|
||||||
|
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
|
||||||
|
// Continue looking for more mount points
|
||||||
|
mp_flag = FindNextVolumeMountPoint(mp_h, mp_buf, MAX_PATH);
|
||||||
|
@@ -2628,7 +2628,7 @@ psutil_disk_partitions(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
goto next;
|
||||||
|
|
||||||
|
next:
|
||||||
|
@@ -2758,9 +2758,9 @@ psutil_users(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_XDECREF(py_username);
|
||||||
|
- Py_XDECREF(py_address);
|
||||||
|
- Py_XDECREF(py_tuple);
|
||||||
|
+ Py_CLEAR(py_username);
|
||||||
|
+ Py_CLEAR(py_address);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
}
|
||||||
|
|
||||||
|
WTSFreeMemory(sessions);
|
||||||
|
@@ -2992,8 +2992,8 @@ psutil_proc_memory_maps(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
- Py_DECREF(py_str);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
+ Py_CLEAR(py_str);
|
||||||
|
}
|
||||||
|
previousAllocationBase = basicInfo.AllocationBase;
|
||||||
|
baseAddress = (PCHAR)baseAddress + basicInfo.RegionSize;
|
||||||
|
@@ -3043,8 +3043,8 @@ psutil_ppid_map(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItem(py_retdict, py_pid, py_ppid))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_pid);
|
||||||
|
- Py_DECREF(py_ppid);
|
||||||
|
+ Py_CLEAR(py_pid);
|
||||||
|
+ Py_CLEAR(py_ppid);
|
||||||
|
} while (Process32Next(handle, &pe));
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -3147,8 +3147,8 @@ psutil_net_if_addrs(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
- Py_DECREF(py_mac_address);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
+ Py_CLEAR(py_mac_address);
|
||||||
|
}
|
||||||
|
|
||||||
|
// find out the IP address associated with the NIC
|
||||||
|
@@ -3224,14 +3224,14 @@ psutil_net_if_addrs(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyList_Append(py_retlist, py_tuple))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_tuple);
|
||||||
|
- Py_DECREF(py_address);
|
||||||
|
- Py_DECREF(py_netmask);
|
||||||
|
+ Py_CLEAR(py_tuple);
|
||||||
|
+ Py_CLEAR(py_address);
|
||||||
|
+ Py_CLEAR(py_netmask);
|
||||||
|
|
||||||
|
pUnicast = pUnicast->Next;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
- Py_DECREF(py_nic_name);
|
||||||
|
+ Py_CLEAR(py_nic_name);
|
||||||
|
pCurrAddresses = pCurrAddresses->Next;
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -3351,8 +3351,8 @@ psutil_net_if_stats(PyObject *self, PyObject *args) {
|
||||||
|
goto error;
|
||||||
|
if (PyDict_SetItem(py_retdict, py_nic_name, py_ifc_info))
|
||||||
|
goto error;
|
||||||
|
- Py_DECREF(py_nic_name);
|
||||||
|
- Py_DECREF(py_ifc_info);
|
||||||
|
+ Py_CLEAR(py_nic_name);
|
||||||
|
+ Py_CLEAR(py_ifc_info);
|
||||||
|
}
|
||||||
|
|
||||||
|
free(pIfTable);
|
@ -1,25 +0,0 @@
|
|||||||
From 4ffbcb458ff0b838f2bfe387b6305c4817e363ae Mon Sep 17 00:00:00 2001
|
|
||||||
From: =?UTF-8?q?Miro=20Hron=C4=8Dok?= <miro@hroncok.cz>
|
|
||||||
Date: Thu, 15 Apr 2021 17:07:38 +0200
|
|
||||||
Subject: [PATCH] Add tolerance to
|
|
||||||
test_linux.TestSystemVirtualMemory.test_total
|
|
||||||
|
|
||||||
We see this test as very flaky without tolerance in Fedora and CentOS
|
|
||||||
---
|
|
||||||
psutil/tests/test_linux.py | 3 ++-
|
|
||||||
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/psutil/tests/test_linux.py b/psutil/tests/test_linux.py
|
|
||||||
index 0c6d498c8..f5243c2cd 100755
|
|
||||||
--- a/psutil/tests/test_linux.py
|
|
||||||
+++ b/psutil/tests/test_linux.py
|
|
||||||
@@ -243,7 +243,8 @@ def test_total(self):
|
|
||||||
# self.assertEqual(free_value, psutil_value)
|
|
||||||
vmstat_value = vmstat('total memory') * 1024
|
|
||||||
psutil_value = psutil.virtual_memory().total
|
|
||||||
- self.assertAlmostEqual(vmstat_value, psutil_value)
|
|
||||||
+ self.assertAlmostEqual(
|
|
||||||
+ vmstat_value, psutil_value, delta=TOLERANCE_SYS_MEM)
|
|
||||||
|
|
||||||
@retry_on_failure()
|
|
||||||
def test_used(self):
|
|
@ -1,24 +0,0 @@
|
|||||||
diff --git a/psutil/tests/test_linux.py b/psutil/tests/test_linux.py
|
|
||||||
index c1d0a94..e9914a9 100755
|
|
||||||
--- a/psutil/tests/test_linux.py
|
|
||||||
+++ b/psutil/tests/test_linux.py
|
|
||||||
@@ -924,6 +924,7 @@ class TestLoadAvg(PsutilTestCase):
|
|
||||||
@unittest.skipIf(not LINUX, "LINUX only")
|
|
||||||
class TestSystemNetIfAddrs(PsutilTestCase):
|
|
||||||
|
|
||||||
+ @unittest.skip("Broken in brew")
|
|
||||||
def test_ips(self):
|
|
||||||
for name, addrs in psutil.net_if_addrs().items():
|
|
||||||
for addr in addrs:
|
|
||||||
diff --git a/psutil/tests/test_system.py b/psutil/tests/test_system.py
|
|
||||||
index a55164c..21a04bb 100755
|
|
||||||
--- a/psutil/tests/test_system.py
|
|
||||||
+++ b/psutil/tests/test_system.py
|
|
||||||
@@ -724,6 +724,7 @@ class TestNetAPIs(PsutilTestCase):
|
|
||||||
self.assertEqual(psutil.net_io_counters(pernic=True), {})
|
|
||||||
assert m.called
|
|
||||||
|
|
||||||
+ @unittest.skip("Broken in brew")
|
|
||||||
def test_net_if_addrs(self):
|
|
||||||
nics = psutil.net_if_addrs()
|
|
||||||
assert nics, nics
|
|
@ -1,134 +0,0 @@
|
|||||||
diff --color -Nur psutil-release-5.8.0.orig/psutil/tests/test_linux.py psutil-release-5.8.0/psutil/tests/test_linux.py
|
|
||||||
--- psutil-release-5.8.0.orig/psutil/tests/test_linux.py 2020-12-18 16:45:00.000000000 -0800
|
|
||||||
+++ psutil-release-5.8.0/psutil/tests/test_linux.py 2021-01-10 15:38:31.104890945 -0800
|
|
||||||
@@ -661,14 +661,13 @@
|
|
||||||
value = int(value.split('-')[1]) + 1
|
|
||||||
self.assertEqual(psutil.cpu_count(), value)
|
|
||||||
|
|
||||||
- @unittest.skipIf(not os.path.exists("/sys/devices/system/cpu"),
|
|
||||||
- "/sys/devices/system/cpu does not exist")
|
|
||||||
+ @unittest.skip("Unreliable on mock")
|
|
||||||
def test_against_sysdev_cpu_num(self):
|
|
||||||
ls = os.listdir("/sys/devices/system/cpu")
|
|
||||||
count = len([x for x in ls if re.search(r"cpu\d+$", x) is not None])
|
|
||||||
self.assertEqual(psutil.cpu_count(), count)
|
|
||||||
|
|
||||||
- @unittest.skipIf(not which("nproc"), "nproc utility not available")
|
|
||||||
+ @unittest.skip("Unreliable on mock")
|
|
||||||
def test_against_nproc(self):
|
|
||||||
num = int(sh("nproc --all"))
|
|
||||||
self.assertEqual(psutil.cpu_count(logical=True), num)
|
|
||||||
@@ -713,7 +712,7 @@
|
|
||||||
m.called
|
|
||||||
|
|
||||||
|
|
||||||
-@unittest.skipIf(not LINUX, "LINUX only")
|
|
||||||
+@unittest.skip("Unreliable in mock")
|
|
||||||
class TestSystemCPUCountPhysical(PsutilTestCase):
|
|
||||||
|
|
||||||
@unittest.skipIf(not which("lscpu"), "lscpu utility not available")
|
|
||||||
@@ -745,7 +744,7 @@
|
|
||||||
@unittest.skipIf(not LINUX, "LINUX only")
|
|
||||||
class TestSystemCPUFrequency(PsutilTestCase):
|
|
||||||
|
|
||||||
- @unittest.skipIf(not HAS_CPU_FREQ, "not supported")
|
|
||||||
+ @unittest.skip("Unreliable in mock")
|
|
||||||
def test_emulate_use_second_file(self):
|
|
||||||
# https://github.com/giampaolo/psutil/issues/981
|
|
||||||
def path_exists_mock(path):
|
|
||||||
@@ -759,7 +758,7 @@
|
|
||||||
create=True):
|
|
||||||
assert psutil.cpu_freq()
|
|
||||||
|
|
||||||
- @unittest.skipIf(not HAS_CPU_FREQ, "not supported")
|
|
||||||
+ @unittest.skip("Unreliable in mock")
|
|
||||||
def test_emulate_use_cpuinfo(self):
|
|
||||||
# Emulate a case where /sys/devices/system/cpu/cpufreq* does not
|
|
||||||
# exist and /proc/cpuinfo is used instead.
|
|
||||||
@@ -888,7 +887,7 @@
|
|
||||||
self.assertEqual(freq.current, 200)
|
|
||||||
|
|
||||||
|
|
||||||
-@unittest.skipIf(not LINUX, "LINUX only")
|
|
||||||
+@unittest.skip("Unreliable in mock")
|
|
||||||
class TestSystemCPUStats(PsutilTestCase):
|
|
||||||
|
|
||||||
def test_ctx_switches(self):
|
|
||||||
@@ -2126,13 +2125,14 @@
|
|
||||||
value = tuple(map(int, value.split()[1:4]))
|
|
||||||
self.assertEqual(self.proc.gids(), value)
|
|
||||||
|
|
||||||
- @retry_on_failure()
|
|
||||||
+ @unittest.skip("Unreliable in mock")
|
|
||||||
def test_num_ctx_switches(self):
|
|
||||||
value = self.read_status_file("voluntary_ctxt_switches:")
|
|
||||||
self.assertEqual(self.proc.num_ctx_switches().voluntary, value)
|
|
||||||
value = self.read_status_file("nonvoluntary_ctxt_switches:")
|
|
||||||
self.assertEqual(self.proc.num_ctx_switches().involuntary, value)
|
|
||||||
|
|
||||||
+ @unittest.skip("Unreliable in mock")
|
|
||||||
def test_cpu_affinity(self):
|
|
||||||
value = self.read_status_file("Cpus_allowed_list:")
|
|
||||||
if '-' in str(value):
|
|
||||||
diff --color -Nur psutil-release-5.8.0.orig/psutil/tests/test_misc.py psutil-release-5.8.0/psutil/tests/test_misc.py
|
|
||||||
--- psutil-release-5.8.0.orig/psutil/tests/test_misc.py 2020-12-18 16:45:00.000000000 -0800
|
|
||||||
+++ psutil-release-5.8.0/psutil/tests/test_misc.py 2021-01-10 15:38:31.104890945 -0800
|
|
||||||
@@ -355,7 +355,7 @@
|
|
||||||
|
|
||||||
def test_setup_script(self):
|
|
||||||
setup_py = os.path.join(ROOT_DIR, 'setup.py')
|
|
||||||
- if CI_TESTING and not os.path.exists(setup_py):
|
|
||||||
+ if not os.path.exists(setup_py):
|
|
||||||
return self.skipTest("can't find setup.py")
|
|
||||||
module = import_module_by_path(setup_py)
|
|
||||||
self.assertRaises(SystemExit, module.setup)
|
|
||||||
diff --color -Nur psutil-release-5.8.0.orig/psutil/tests/test_system.py psutil-release-5.8.0/psutil/tests/test_system.py
|
|
||||||
--- psutil-release-5.8.0.orig/psutil/tests/test_system.py 2020-12-18 16:45:00.000000000 -0800
|
|
||||||
+++ psutil-release-5.8.0/psutil/tests/test_system.py 2021-01-10 15:38:31.105890946 -0800
|
|
||||||
@@ -198,7 +198,7 @@
|
|
||||||
self.assertGreater(bt, 0)
|
|
||||||
self.assertLess(bt, time.time())
|
|
||||||
|
|
||||||
- @unittest.skipIf(CI_TESTING and not psutil.users(), "unreliable on CI")
|
|
||||||
+ @unittest.skipIf(not psutil.users(), "unreliable on CI")
|
|
||||||
def test_users(self):
|
|
||||||
users = psutil.users()
|
|
||||||
self.assertNotEqual(users, [])
|
|
||||||
@@ -510,7 +510,7 @@
|
|
||||||
if not AIX and name in ('ctx_switches', 'interrupts'):
|
|
||||||
self.assertGreater(value, 0)
|
|
||||||
|
|
||||||
- @unittest.skipIf(not HAS_CPU_FREQ, "not suported")
|
|
||||||
+ @unittest.skip("Unreliable in mock")
|
|
||||||
def test_cpu_freq(self):
|
|
||||||
def check_ls(ls):
|
|
||||||
for nt in ls:
|
|
||||||
@@ -579,6 +579,7 @@
|
|
||||||
def test_disk_usage_bytes(self):
|
|
||||||
psutil.disk_usage(b'.')
|
|
||||||
|
|
||||||
+ @unittest.skip("unreliable on CI")
|
|
||||||
def test_disk_partitions(self):
|
|
||||||
def check_ntuple(nt):
|
|
||||||
self.assertIsInstance(nt.device, str)
|
|
||||||
@@ -828,7 +829,7 @@
|
|
||||||
|
|
||||||
class TestSensorsAPIs(PsutilTestCase):
|
|
||||||
|
|
||||||
- @unittest.skipIf(not HAS_SENSORS_TEMPERATURES, "not supported")
|
|
||||||
+ @unittest.skip("Unreliable in mock")
|
|
||||||
def test_sensors_temperatures(self):
|
|
||||||
temps = psutil.sensors_temperatures()
|
|
||||||
for name, entries in temps.items():
|
|
||||||
diff --color -Nur psutil-release-5.8.0.orig/psutil/tests/test_testutils.py psutil-release-5.8.0/psutil/tests/test_testutils.py
|
|
||||||
--- psutil-release-5.8.0.orig/psutil/tests/test_testutils.py 2020-12-18 16:45:00.000000000 -0800
|
|
||||||
+++ psutil-release-5.8.0/psutil/tests/test_testutils.py 2021-01-10 15:48:14.240695423 -0800
|
|
||||||
@@ -364,7 +364,7 @@
|
|
||||||
self.assertRaises(ValueError, self.execute, lambda: 0, retries=-1)
|
|
||||||
|
|
||||||
@retry_on_failure()
|
|
||||||
- @unittest.skipIf(CI_TESTING, "skipped on CI")
|
|
||||||
+ @unittest.skip("Unreliable in mock")
|
|
||||||
def test_leak_mem(self):
|
|
||||||
ls = []
|
|
||||||
|
|
Loading…
Reference in new issue