it's very hard to read this code without tabs, but maybe this is an error: > do_upload2($userfile,$newname);
shouldn't that be $userfile2 and $newname2 ? (if this is the error, now you see why ugly code sucks :)) anyway, maybe we would known much more if we saw the whole code, e.g. where CurrentImage gets initialized or the form that submits the files -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php