Uploadscript - Wo liegt das Problem

kojak · 1. November 2004

Hi,

ich hab mit ein Script gebastelt damit meine Besucher Screenshots hochladen können. Leider funktioniert es irgendwie nicht so richtig. Ich bekomme jedesmal die Fehlermeldung, dass die Datei zu groß ist obwohl sie es gar nicht ist:

PHP

<?php
require('config/password.php');
include("config/indexinclude.php");
$MAX_FILE_SIZE = $_REQUEST['MAX_FILE_SIZE'];
$bid = $_REQUEST['bid'];
$ueberschrift = $_REQUEST['ueberschrift'];
$uploadedfile = $_REQUEST['uploadedfile'];
$submit = $_REQUEST['submit'];
?>
<table border='0' width='60%' bgcolor='d3e2ea' cellpadding='0' cellspacing='0' align='center'>
<tr valign='top'>
<td style='height: 20px;' width='20'><img src='pics/ecke_lo.gif' alt='ecke links oben'></td>
<td style='background: url(pics/oben.gif);'></td>
<td style='height: 20px;' width='20'><img src='pics/ecke_ro.gif' alt='ecke rechts oben'></td>
</tr>
<tr valign='top'>
<td style='background: url(pics/links.gif);'></td>
<td align='center'>
<b><big><u>UEBERSCHRIFT</u></big></b><br>
<br>
<br>
<?php
if ( "image/jpeg" == $uploadedfile_type
or "image/gif" == $uploadedfile_type
or "image/png" == $uploadedfile_type)
{
if ( "image/jpeg" == $uploadedfile_type )
{
$extension = ".jpg";
}
if ( "image/png" == $uploadedfile_type )
{
$extension = ".png";
}
if ( "image/gif" == $uploadedfile_type )
{
$extension = ".gif";
}
$filename = "screenshots/" . $ueberschrift . time() . $uploadedfile_size . $extension;
if (is_uploaded_file($uploadedfile) &&
copy($uploadedfile, $filename))
{ echo(" $filename Datei erfolgreich als Screenshot 3 gespeichert ");
// Dateinamen in der Datenbank speichern
include ("config/config.inc");
mysql_select_db("$database");
$sql = "UPDATE Artikel SET SCREENSHOT3='$filename' WHERE ID='$bid'";
if (@mysql_query($sql)) {
echo("<p>Neuer Screenshot hinzugefgt</p>");
}
else {
echo("<p>Fehler beim Hinzufügen des Screenshots: " .
mysql_error() . "</p>");
}
// Weiteren Screenshot hinzufgen:
echo ("<br><br>Einen 4. Screenshot hinzufgen:<br><br>");
echo("
<br>
<br>
<form action='fileupload4.php' method='post' enctype='multipart/form-data'>
<input type='hidden' name='MAX_FILE_SIZE' value='120000'>
<input type='hidden' name='bid' value='$bid'>
<p>Screenshot3 <input type='file' name='uploadedfile'></p>
<p><input type='submit' name='submit' value='Hochladen'>
</p>
</form><br>
<br>
Oder:<br>");
}
else { echo(" $filename konnte nicht gespeichert werden. Bitte wenden Sie sich an einen Administrator");}
}
else
{
echo ("Bitte laden Sie nur jpeg, gif oder .png Dateien hoch die unter 117 KB sind!!!");
}
echo("<b><big><a href='berichtentsperren.php?id=$bid'>Bericht freigeben</a></big></b>");
?>
<td style='background: url(pics/rechts.gif);'></td>
</tr>
<tr valign='top'>
<td style='height: 20px;' width='20'><img src='pics/ecke_lu.gif' alt='ecke links unten'></td>
<td style='background: url(pics/unten.gif);'></td>
<td style='height: 20px;' width='20'><img src='pics/ecke_ru.gif' alt='ecke rechts unten'></td>
</table>
<br>
<br>
</td>
</tr>
</table>
</body>
</html>

Alles anzeigen

Irgendwelche Ideen?

Danke
kojak

Dene · 1. November 2004

Ich würd einfach eins ohne MySQL nehmen. Es gibt doch genügend solche Scripts im Web - wenn eins nicht geht, einfach nächstes nehmen.

Aber falls du unbedingt das obere brauchst: welchen hoster hast du? Nen freespace?

Dennis The Menace · 2. November 2004

Mal eine ganz stumpfe Frage:

Meldet PHP/Apache denn, dass die Datei zu groß ist, oder meinst du deinen
eigenen echo Befehl?

PHP

<?php
echo ("Bitte laden Sie nur jpeg, gif oder .png Dateien hoch die unter 117 KB sind!!!");
?>

BTW.: Du musst nicht bei jedem echo, include ... klammern rummachen
echo "Hello World"; reicht auch schon

the-architect · 2. November 2004

die maximale dateigröße kann auch in der php.ini eingestellt werden. hast du da schonmal geschaut?

http://de.php.net/echo

Zitat

echo() ist keine Funktion, sondern ein Sprach-Konstrukt. Deshalb brauchen Sie keine Klammern zu verwenden. Wenn Sie echo mehr als einen Parameter übergeben wollen, dürfen Sie diese nicht mit Klammern umgeben. Es ist nicht möglich, echo() in einem Variablen-Funktionen Kontext zu nutzen.

kojak · 3. November 2004

Zitat

Meldet PHP/Apache denn, dass die Datei zu groß ist, oder meinst du deinen
eigenen echo Befehl?

Er bringt mir meinen echo Befehl.

Zitat

Ich würd einfach eins ohne MySQL nehmen.

Es geht mir ja nicht einfach nur darum, dass die Datei hochgeladen wird, sondern sie wird direkt an der richtigen Stelle in meine Internetseite integriert. Ich wüsste nicht wie ich das ohne MYSQL machen könnte.

danke
kojak

loki · 3. November 2004

Du fragst doch gar nicht die Größe abe, sondern wenn überhaupt nur den Filetype..
Das kann so gar nicht funktionieren

kojak · 3. November 2004

Schau mal beim Formular Neuen Screenshot hinzufügen unterhalb des eigentlichen Scripts:

Code

<input type='hidden' name='MAX_FILE_SIZE' value='120000'>

mfg
kojak

loki · 3. November 2004

Du fragst das aber nirgendwo im PHP ab!
Deine if Abfrage, die nachher den Größenfehler ausgibt ist nur auf Bildertypen bestimmt, nicht aber auf eine Größe!

Und wie das im Form aussieht, keine ahnung ob das so funktioniert, habe das noch nie gesehen, dass das ganze so eine zu große Datei abfängt..

kojak · 3. November 2004

Hi,

ok ich hab den Fehler etwas eingeschränkt:

PHP

if ( "image/jpeg" == $uploadedfile_type
or "image/gif" == $uploadedfile_type
or "image/png" == $uploadedfile_type)

Er scheint nicht zu erkennen, welcher filetype die Datei ist. Mit der Größe hat das letztendlich nichts zu tun. Wie bekomme ich ihn dazu die Variable $uploadedfile_type zu erkennen?
In dem Buch aus dem ich dieses Script habe, steht, dass er die Variablen selbst erstellt. Muss ich am Anfang z.B.
$uploadedfile_type = $_SERVER['uploadedfile_type'];
einfügen?

mfg
kojak

wys · 3. November 2004

Zitat von kojak

Schau mal beim Formular Neuen Screenshot hinzufügen unterhalb des eigentlichen Scripts:

Code

<input type='hidden' name='MAX_FILE_SIZE' value='120000'>

Vergiss den Quatsch mit MAX_FILE_SIZE, ist sehr unsicher, hängt einfach davon ab, ob der Browser es auch umsetzt.
loki hat schon völlig recht, eine sichere Abfrage erreichst Du nur serverseitig, z.B. per PHP.

Gruß ...
wys

kojak · 3. November 2004

Hi,

ich habs bereits rausgenommen, aber das ändert an meinem Problem mit den Variablen nix.

Danke
kojak

kojak · 3. November 2004

Hier noch etwas konkreter. Folgende Variablen enthalten aus welchen Gründen auch immer keine Werte:

uploadedfile_type
$uploadedfile_size
$uploadedfile_name
$uploadedfile

Woran kann es liegen, dass diese nicht definiert sind? Auch meinem alten Server hat die immer wunderbar funktioniert. Aber seitdem mein Provider umgezogen ist haut das einfach nicht mehr hin. Es wurde irgendetwas an den register_globals geändert.

danke
kojak

loki · 3. November 2004

Versuchs mal mit der superglobal $_FILES[]

PHP

$_FILES['uploadfeldname']['name']
$_FILES['uploadfeldname']['size']
$_FILES['uploadfeldname']['type']

kojak · 3. November 2004

Hi,

vielen vielen danke, das scheint soweit zu funktionieren. Nur noch ein kleines Problemchen:
Die Datei wird ja in einem Temporären Verzeichnis mit einem Temorären Namen zwischengespeichert, bevor ich sie z.B. mit "move_uploaded_file" verschieben kann. Um sie zu verschieben benötige ich jedoch diesen temporären Namen, wie erhalte ich diesen?

danke
RS

Dennis The Menace · 3. November 2004

$_FILES['uploadfeldname']['tmp_name']

wenn ich raten müsste

kojak · 3. November 2004

habs auch gerade in der doku gefunden. danke

loki · 3. November 2004

Dann verteil mal ein paar renomee punkte

kojak · 3. November 2004

Hi,

es funktioniert jetzt. 1000 Dank. Ohne euch hätte ich es wohl nie geschafft.

Mfg
kojak

Erdferkel · 13. Februar 2005

um dieses thema wieder aufleben zu lassen:

ich uploade ein photo mit hilfe eines formulars und move_uploaded_file:

PHP

$_FILES['photo']['tmp_name'], '../images/'.$_FILES['photo']['name']

es klappt alles wunderbar, solange ich als zielpfad '../images/' eingebe, will ich aber die datei in '../images/band/' abspeichern, bekomme ich eine fehlermeldung:

Warning: move_uploaded_file(../images/band/000.jpg): failed to open stream: No such file or directory in /usr/export/www/vhosts/funnetwork/hosting/mentalservice/admin/band.php on line 15

Warning: move_uploaded_file(): Unable to move '/usr/export/tmp/php2KrQGR' to '../images/band/000.jpg' in /usr/export/www/vhosts/funnetwork/hosting/mentalservice/admin/band.php on line 15

das unterverzeichnis band existiert wohlbemerkt.

was mir aber auffällt - ich kann nirgendwo auf dem webspace (via ftp mit iexplorer) interne "kopieren" und "einfügen" aktionen machen. warum klappt dann aber überhaupt erst die aktion mit '../images/' ?

Teilen