giampaolo / psutil

92%

LAST BUILD ON BRANCH master

branch: master

CHANGE BRANCH

x

Reset

• master
• release-3.1.0
• 655-windows-unicode
• spacewander-patch-1
• release-3.2.0
• release-3.2.1
• landryb-openbsd
• release-3.2.2
• EvinceMoi-fix-net_io_counter-overflow
• release-3.3.0
• 613-thread-names
• 720-sysinfo
• 0-wiz-0-master
• release-3.4.1
• release-3.4.2
• fbenkstein-non-unicode
• revive-environ-windows
• bsebi-master
• mem-refactoring
• 756-disk-io-busy-time
• 777-linux-files-pos-mode
• 779-proc-cpu-children-times
• 722-net-io-ext
• mem-refactoring-2
• 772-net-io-counters-ext
• 795-win-services
• release-4.1.0
• oneshot
• appveyor-experiments
• release-4.2.0
• release-4.3.0
• oneshot-win
• release-4.3.1
• 887-linux-free-mem-standardization
• revert-888-887-linux-free-mem-standardization
• sync-test-primitives
• refactor-c-pid-exists
• 887-linux-free-mem-standardization-2
• osx-proc-pidinfo-refact
• release-4.4.0
• net-if-stats-refact
• release-4.4.1
• release-4.4.2
• release-5.0.0
• 357-proc-cpu-num
• 941-cpu-freq
• release-5.0.1
• 955-battery-sensors
• 941-cpu-info
• release-5.1.1
• release-5.1.2
• release-5.1.3
• 922-if-flags
• nicolargo-fansensor
• release-5.2.0
• release-5.2.1
• release-5.2.2
• 1018-test-refactoring
• 1040-fix-unicode
• 1053-drop-py26-test-support
• 802-wrap-numbers
• 1058-enable-fix-warnings
• 930-netbsd-connections-segfault
• psutil-testing-envvar
• win-openprocess-fix-2
• fix-1091-nitpicks
• alxchk-master
• 1044-osx-zombies
• 1102-proc-bitness
• release-5.3.0
• release-5.3.1
• release-5.4.0
• osx-travis-failure
• 1151-PSUTIL_TESTING_env_var
• 1173-debug-mode
• c-excs-refact
• 1183-proc-children-speedup
• debug-travis-osx
• future-warning
• set_procfs_path
• release-5.4.2
• osx-pids-fix
• osx-wheels-debug
• open-mock-refactoring
• appveyor-debug
• 771-win-cpu-count
• release-5.4.4
• release-5.4.5
• release-5.4.6
• osx-task-for-pid
• 1286-rename-osx-macos
• linux-disk-io-partitions
• osx-temps-alternate
• release-5.4.7
• thread-safety
• thread-safety-2
• win-openprocess-access
• release-5.5.0
• release-5.5.1
• win-alternate-exe-api
• win-c-refactoring
• win-loadlib-refact
• winloadlib
• release-gil
• rich-exceptions
• fix-win-set-debug
• win-proc-suspend-resume
• osx-mem-maps-segfault
• fix-ntext.h
• win7
• dragonfly-bsd
• release-5.6.0
• win-uss-mem
• release-5.6.1
• big-docfix
• fix-tests
• win-getloadavg
• cygwin
• revert-1493-fix_cpu_freq
• tidelift
• release-5.6.3
• connections-refactoring
• fs-exceptions
• c-mod-init
• release-5.6.4
• release-5.6.5

Build # 3069

Build Type

push

travis-ci

Committed by giampaolo

Commit Message

Use Py_CLEAR instead of Py_DECREF to also set the variable to NULL (#1616)

These files contain loops that convert system data into python objects
and during the process they create objects and dereference their
refcounts after they have been added to the resulting list.

However, in case of errors during the creation of those python objects,
the refcount to previously allocated objects is dropped again with
Py_XDECREF, which should be a no-op in case the paramater is NULL. Even
so, in most of these loops the variables pointing to the objects are
never set to NULL, even after Py_DECREF is called at the end of the loop
iteration. This means, after the first iteration, if an error occurs
those python objects will get their refcount dropped two times,
resulting in a possible double-free.

Run Details

2148 of 2342 relevant lines covered (91.72%)

0.92 hits per line