Arch Linux

Kniffliges Bash-Problem....

[gelöscht]

Hallo,

ich würde gerne mittel Bash-Script und youtube-dl ein gewünschtes File von dort holen, es automatisch mit ffmpeg zu avi umkonvertieren und in ein Verzeichnis verschieben.
Das Problem ist aber, dass das File, wenn ich es auf diese Weise herunterlade, offensichtlich irgendwie zerstört ist. Hier das Script:

#!/bin/bash

if [ $# -ge 2 ]; then
echo "Downloading $1..."
FILE=$(youtube-dl $1)
echo "Converting $1 to $2.avi..."
DEST=$(ffmpeg -i $FILE -sameq -aspect 4:3 -acodec mp3 $2.avi)
mv $DEST /mnt/multimedia/YouTube-Videos/$DEST
else
echo "Usage: ytdownloader YOUTUBE-URL TARGET-FILENAME"
fi

welches zu dieser Ausgabe führt .....

markus@gentoo ~ $ ./ytdownloader http://www.youtube.com/watch?v=gsWqfxld9e8 Königin
Downloading http://www.youtube.com/watch?v=gsWqfxld9e8...
Converting http://www.youtube.com/watch?v=gsWqfxld9e8 to Königin.avi...
FFmpeg version SVN-rUNKNOWN, Copyright (c) 2000-2007 Fabrice Bellard, et al.
configuration: --prefix=/usr --libdir=/usr/lib --shlibdir=/usr/lib --mandir=/usr/share/man --enable-static --enable-shared --cc=i686-pc-linux-gnu-gcc --disable-altivec --disable-debug --disable-audio-oss --disable-v4l --disable-v4l2 --disable-dv1394 --disable-network --disable-opts --enable-libmp3lame --enable-libvorbis --enable-libogg --enable-xvid --enable-x11grab --enable-libogg --enable-libfaad --enable-libfaac --enable-gpl --enable-pp --disable-strip
libavutil version: 49.4.0
libavcodec version: 51.40.2
libavformat version: 51.11.0
built on May 17 2007 10:42:42, gcc: 4.1.1 (Gentoo 4.1.1-r3)
Retrieving: I/O error occured
Usually that means that input file is truncated and/or corrupted.

Was kann ich tun?

[gelöscht]

ori schrieb FILE=$(youtube-dl $1)
DEST=$(ffmpeg -i $FILE -sameq -aspect 4:3 -acodec mp3 $2.avi)

Diese zwei Zeilen funktionieren nicht so wie du es wolltest.

Der Befehl
VAR=$(befehl)
speichert die Ausgabe des Befehls 'befehl' in der Variable VAR. youtube-dl gibt aber nicht nur den Dateiname aus, sondern noch einige andere Informationen.

Ich würde den -o Parameter nutzen:

youtube-dl $1 -o $2.flv
ffmpeg -i $2.flv -sameq -aspect 4:3 -acodec mp3 $2.avi
mv $2.avi /pfad/zum/zielverzeichnis/
rm $2.flv

[gelöscht]

Vielen Dank, so klappt es.
Daran dachte ich ja gar nicht, dass die Ausgabe ja auch aus dem Text besteht.
Aber eine Frage dazu:

Würde von youtube-dl kein Text ausgegeben werden, würde die Variable FILE dann ein gültiges Flash-Movie beinhalten?

[gelöscht]

ori schrieb Würde von youtube-dl kein Text ausgegeben werden, würde die Variable FILE dann ein gültiges Flash-Movie beinhalten?

Wenn youtube-dl nur den Dateinamen ausgibt, dann enthält die Variable FILE den Dateinamen und man kann mit diesem weiterarbeiten (z. B. ansehen mit "mplayer $FILE").
Wenn youtube-dl das Flash-Movie direkt auf der Standardausgabe ausgeben würde (als Buchstabensalat), dann würde die Variable FILE ein gültiges Flash-Movie enthalten. Man könnte es mit "echo -n $FILE | mplayer" ansehen.
Wenn youtube-dl kein Text (zu deutsch nichts) ausgibt, ist die Variable $FILE leer

EDIT: Ausgeben heißt hier immer auf der Standardausgabe ausgeben. Das ist meist der Bildschirm oder bei "VAR=$(befehl)" eben die Variable. Andere Ausgabedateien sind davon völlig unberührt.

[gelöscht]

Ok alles klar. Eines fiel mir allerdings noch auf. Wenn der auszugebende Name mehr als ein Wort hat, kann offensichtlich mit dem Leerzeichen-Escapen nicht gearbeitet werden (Fehlermeldungen), als auch nicht das Wort ohne diese Escape-Slashes eingegeben werden, da dann logischerweise nur das erste Wort ($2) verarbeitet würde.
Ich denke mir, dass es mit "shift" und $* zu bewerkstelligen ist, aber könnte das ein Sicherheitsrisiko darstellen? Theoretisch könnte man ja auf diese Weise beliebig viel und weiß der Kuckuck welchen Text einschleusen, zB

ytdownloader URL Ich bin der böse String, der Bufferüberläufe o.ä. erzeugt

[gelöscht]

Du hast recht. Dateinamen mit Leerzeichen verarbeitet das Script noch nicht. Dazu muss man die Variablennamen in Anführungszeichen setzen:

youtube-dl "$1" -o "$2.flv"
ffmpeg -i "$2.flv" -sameq -aspect 4:3 -acodec mp3 "$2.avi"
mv "$2.avi" /pfad/zum/zielverzeichnis/
rm "$2.flv"

Ein Sicherheitsrisiko dürfte das meines Wissens nicht sein. Zum einen gibt es eine Beschrenkung der Bash (ich glaube ~65536). Zum anderen sind Sicherheitslücken immer Programmfehler (meist in C(++)-Programmen). Ein Bash oder Python-Programm ist da nicht anfällig.

[gelöscht]

Danke so klappt es 🙂