Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 15 Jun 2021 00:41:53 +0000
From:      bugzilla-noreply@freebsd.org
To:        ports-bugs@FreeBSD.org
Subject:   [Bug 256612] multimedia/plexmediaserver: rc.d file uncleanly tries to kill nonexistent process
Message-ID:  <bug-256612-7788@https.bugs.freebsd.org/bugzilla/>

next in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D256612

            Bug ID: 256612
           Summary: multimedia/plexmediaserver: rc.d file uncleanly tries
                    to kill nonexistent process
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Some People
          Priority: ---
         Component: Individual Port(s)
          Assignee: feld@FreeBSD.org
          Reporter: blue.esper@icloud.com
          Assignee: feld@FreeBSD.org
             Flags: maintainer-feedback?(feld@FreeBSD.org)

the plex_stop_postcmd function in /usr/local/etc/rc.d/plexmediaserver attem=
pts
to kill any leftover child processes, but the kill command runs whether the=
re
are any or not. This makes it run without input if there are no processes to
kill, making it return an error if you restart the plexmediaserver service.
It's benign but unsightly, and I think it should be adjusted a bit to test =
if
the variable is empty before trying to kill it. Something to the effect of:


        if [ -z ${_PLEX_CHILDREN} ];then
            kill -9 ${_PLEX_CHILDREN}
        fi

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-256612-7788>