accepter les documents dont le nom contient deux .. (mais changer leur nom), resoud #978

ecrire/inc/ajouter_documents.php

@@ -49,7 +49,7 @@ function ajouter_un_document($source, $nom_envoye, $type_lien, $id_lien, $mode,
 // Documents distants : pas trop de verifications bloquantes, mais un test
 // via une requete HEAD pour savoir si la ressource existe (non 404), si le
 // content-type est connu, et si possible recuperer la taille, voire plus.
-	spip_log ("ajout du document $nom_envoye  (M '$mode' T '$type_lien' L '$id_lien' D '$id_document')");
+	spip_log ("ajout du document $source $nom_envoye  (M '$mode' T '$type_lien' L '$id_lien' D '$id_document')");
 	if ($mode == 'distant') {
 		include_spip('inc/distant');
 		if ($a = recuperer_infos_distantes($source)) {
@@ -123,7 +123,9 @@ function ajouter_un_document($source, $nom_envoye, $type_lien, $id_lien, $mode,
 			$fichier = copier_document($ext, $nom_envoye, $source);
 			spip_unlink($source);
 
-		} else $fichier = copier_document($ext, $nom_envoye, $source);
+		} else {
+			$fichier = copier_document($ext, $nom_envoye, $source);
+		}
 
 		// Verifier que le fichier est a son emplacement definitif
 		

ecrire/inc/getdocument.php

@@ -51,6 +51,7 @@ function effacer_repertoire_temporaire($nom) {
 // http://doc.spip.org/@copier_document
 function copier_document($ext, $orig, $source) {
 
+	$orig = preg_replace(',\.\.+,', '.', $orig); // pas de .. dans le nom du doc
 	$dir = creer_repertoire_documents($ext);
 	$dest = preg_replace("/[^._=-\w\d]+/", "_", 
 			translitteration(preg_replace("/\.([^.]+)$/", "", 
@@ -68,7 +69,7 @@ function copier_document($ext, $orig, $source) {
 	$n = 0;
 	while (@file_exists($newFile = $dir . $dest .($n++ ? ('-'.$n) : '').'.'.$ext));
 
-	return (deplacer_fichier_upload($source, $newFile)) ? $newFile : '';
+	return deplacer_fichier_upload($source, $newFile);
 }
 
 //
@@ -78,11 +79,7 @@ function copier_document($ext, $orig, $source) {
 // http://doc.spip.org/@deplacer_fichier_upload
 function deplacer_fichier_upload($source, $dest, $move=false) {
 	// Securite
-	## !! interdit pour le moment d'uploader depuis l'espace prive (UPLOAD_DIRECT)
-	if (strstr($dest, "..")) {
-		spip_log("stop deplacer_fichier_upload: '$dest'");
-		exit;
-	}
+	$dest = preg_replace(',\.\.+,', '.', $dest);
 
 	if ($move)	$ok = @rename($source, $dest);
 	else				$ok = @copy($source, $dest);
@@ -99,7 +96,7 @@ function deplacer_fichier_upload($source, $dest, $move=false) {
 		}
 		spip_unlink($dest);
 	}
-	return $ok;
+	return $ok ? $dest : false;
 }