* Dylan Aïssi <dai...@debian.org>, 2020-07-06, 15:13:
pytmpdir=$(mktemp -d "${TMPDIR:-/tmp}"/XXXXXXXXXXXX) pywrapper=$(mktemp "${pytmpdir}"/bashtop.psutil.XXXX)Should fix the issue.Does the proposed fix by upstream look good to you?
It's mostly OK, but:
- mktemp(1) can fail, and this failure should not be ignored;
- there's no need for two mktemp calls.
Something like this should work:
pytmpdir=$(mktemp -d "${TMPDIR:-/tmp}"/bashtop.XXXXXX) || exit 1
pywrapper=${pytmpdir}/psutil
--
Jakub Wilk
