From 3110662020a0144fc65d7be9304e4fd424f5d234 Mon Sep 17 00:00:00 2001
From: Rouji <rj@x0.at>
Date: Mon, 2 Oct 2017 20:56:04 +0200
Subject: [PATCH] up the filename length only after multiple collisions

---
 index.php | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/index.php b/index.php
index 198d060..89b5117 100755
--- a/index.php
+++ b/index.php
@@ -116,13 +116,20 @@ function storeFile($name, $tmpFile, $formatted = false)
     }
 
     $ext = getExtension($name);
-    $len = $ID_LENGTH;
-    do //generate filenames until we get one, that doesn't already exist
+    $tries_per_len=3; //try random names a few times before upping the length
+    for ($len = $ID_LENGTH; ; ++$len)
     {
-        $id = rndStr($len++);
-        $basename = $id . (empty($ext) ? '' : '.' . $ext);
-        $target_file = $STORE_PATH . $basename;
-    } while (file_exists($target_file));
+        echo $len;
+        for ($n=0; $n<=$tries_per_len; ++$n)
+        {
+            $id = rndStr($len);
+            $basename = $id . (empty($ext) ? '' : '.' . $ext);
+            $target_file = $STORE_PATH . $basename;
+
+            if (!file_exists($target_file))
+                break 2;
+        }
+    }
 
     $res = move_uploaded_file($tmpFile, $target_file);
     if ($res)