[BUG] TMPDIR not quoted in init-plex-chown/run script

Offen Anfängerfreundlich
#458 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Anfängerfreundlichkeit
92/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
docker, shell

Rechercherichtung

Öffne root/etc/s6-overlay/s6-rc.d/init-plex-chown/run und vergleiche die nicht in Anführungszeichen gesetzte TMPDIR-Verwendung in Zeile 8 mit der in Anführungszeichen gesetzten Verwendung in Zeile 6. Überprüfe das Skript mit einem TMPDIR-Pfad, der Leerzeichen enthält; die Aufgabe ist abgeschlossen, wenn mkdir diesen Pfad korrekt verarbeitet, ohne falsche Verzeichnisse zu erstellen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

no-issue-activity

Current Behavior

In root/etc/s6-overlay/s6-rc.d/init-plex-chown/run, line 8 uses mkdir -p ${TMPDIR} without quoting the variable. While the default value (/run/plex-temp) doesn't contain spaces, if a user sets TMPDIR to a path with spaces, the mkdir command will fail or create the wrong directory.

Expected Behavior

TMPDIR should be quoted consistently, like it is on line 6 where rm -rf "${TMPDIR}" is used.

Steps To Reproduce

  1. Set TMPDIR=/path with spaces/plex-temp in docker create/run command
  2. Start the container
  3. The mkdir -p command will fail or create incorrect directories

Environment

  • OS: Any
  • How docker service was installed: Any
  • CPU architecture: x86-64 or arm64
  • Docker creation: Any docker create/run command with TMPDIR containing spaces

Container logs

N/A - this is a code quality issue

Vorherrschende Sprache
Dockerfile
Sterne
1.4k
Forks
228
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus linuxserver/docker-plex

Alle Issues in linuxserver/docker-plex

Ähnliche Issues

Weitere Issues zu DevOps

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.