# Buggy code
def exe(self):
    try:
        exe = os.readlink("/proc/%s/exe" % self.pid)
    except (OSError, IOError) as err:
        if err.errno in (errno.ENOENT, errno.ESRCH):
            # no such file error; might be raised also if the
            # path actually exists for system processes with
            # low pids (about 0-20)
            if os.path.lexists("/proc/%s" % self.pid):
                return ""
            else:
                if not pid_exists(self.pid):
                    raise NoSuchProcess(self.pid, self._name)
                else:
                    raise ZombieProcess(self.pid, self._name)
        if err.errno in (errno.EPERM, errno.EACCES):
            raise AccessDenied(self.pid, self._name)
        raise

    # readlink() might return paths containing null bytes ('\x00').
    # Certain names have ' (deleted)' appended. Usually this is
    # bogus as the file actually exists. Either way that's not
    # important as we don't want to discriminate executables which
    # have been deleted.
    exe = exe.split("\x00")[0]
    if exe.endswith(" (deleted)") and not os.path.exists(exe):
        exe = exe[:-10]
    return exe


# Traceback (psutil._error.NoSuchProcess)
How to reproduce:

1. start Photoshop CS6 on a Mountain Lion OSX
2. import psutil; [x.as_dict() for x in psutil.process_iter()] # (in .py file, ipython)



What is the expected output?
A long list of processes and related information



What do you see instead?


$ python test.py
Traceback (most recent call last):
File "test.py", line 3, in <module>
[x.as_dict() for x in psutil.process_iter() if x.is_running()]
File
"/Users/rico/.virtualenvs/temp/lib/python2.7/site-packages/psutil/__init__.py",
line 225, in as_dict
ret = attr()
File
"/Users/rico/.virtualenvs/temp/lib/python2.7/site-packages/psutil/__init__.py",
line 414, in get_nice
return self._platform_impl.get_process_nice()
File
"/Users/rico/.virtualenvs/temp/lib/python2.7/site-packages/psutil/_psosx.py",
line 151, in wrapper
raise NoSuchProcess(self.pid, self._process_name)
psutil._error.NoSuchProcess: process no longer exists (pid=46244)

or within iPython notebook:
[x.as_dict() for x in psutil.process_iter() if x.is_running()]
---------------------------------------------------------------------------
NoSuchProcess                             Traceback (most recent call last)
<ipython-input-108-a71c6dffe397> in <module>()
----> 1 [x.as_dict() for x in psutil.process_iter() if x.is_running()]

/Users/rico/.virtualenvs/temp/lib/python2.7/site-packages/psutil/__init__.pyc
in as_dict(self, attrs, ad_value)
223                         ret = attr(interval=0)
224                     else:
--> 225                         ret = attr()
226                 else:
227                     ret = attr

/Users/rico/.virtualenvs/temp/lib/python2.7/site-packages/psutil/__init__.pyc
in get_nice(self)
412     def get_nice(self):
413         """Get process niceness (priority)."""
--> 414         return self._platform_impl.get_process_nice()
415
416     @_assert_pid_not_reused

/Users/rico/.virtualenvs/temp/lib/python2.7/site-packages/psutil/_psosx.pyc in
wrapper(self, *args, **kwargs)
149             err = sys.exc_info()[1]
150             if err.errno == errno.ESRCH:
--> 151                 raise NoSuchProcess(self.pid, self._process_name)
152             if err.errno in (errno.EPERM, errno.EACCES):
153                 raise AccessDenied(self.pid, self._process_name)

NoSuchProcess: process no longer exists (pid=46243, name='adobe_licutil')

When I close Photoshop, the error will not show up. When starting it again the
error reappears.
An additional is_running() check within the list comprehension does not change
a thing and running the code several times will not change the reported pid.

# Fixed code
def exe(self):
    try:
        exe = os.readlink("/proc/%s/exe" % self.pid)
    except (OSError, IOError) as err:
        if err.errno in (errno.ENOENT, errno.ESRCH):
            # no such file error; might be raised also if the
            # path actually exists for system processes with
            # low pids (about 0-20)
            if os.path.lexists("/proc/%s" % self.pid):
                return ""
            else:
                if not pid_exists(self.pid):
                    raise NoSuchProcess(self.pid, self._name)
                else:
                    raise ZombieProcess(self.pid, self._name, self._ppid)
        if err.errno in (errno.EPERM, errno.EACCES):
            raise AccessDenied(self.pid, self._name)
        raise

    # readlink() might return paths containing null bytes ('\x00').
    # Certain names have ' (deleted)' appended. Usually this is
    # bogus as the file actually exists. Either way that's not
    # important as we don't want to discriminate executables which
    # have been deleted.
    exe = exe.split("\x00")[0]
    if exe.endswith(" (deleted)") and not os.path.exists(exe):
        exe = exe[:-10]
    return exe