Index: branches/5.2.x/core/units/helpers/file_helper.php
===================================================================
diff -u -N -r15173 -r15533
--- branches/5.2.x/core/units/helpers/file_helper.php	(.../file_helper.php)	(revision 15173)
+++ branches/5.2.x/core/units/helpers/file_helper.php	(.../file_helper.php)	(revision 15533)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: file_helper.php 15173 2012-03-09 12:35:38Z alex $
+* @version	$Id: file_helper.php 15533 2012-09-11 16:25:31Z alex $
 * @package	In-Portal
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -423,7 +423,7 @@
 		 * @param Array $forbidden_names
 		 * @return string
 		 */
-		public function ensureUniqueFilename($path, $name, $forbidden_names = Array())
+		public function ensureUniqueFilename($path, $name, $forbidden_names = Array ())
 		{
 			$parts = pathinfo($name);
 			$ext = '.' . $parts['extension'];
@@ -441,7 +441,7 @@
 			// when storage engine folder is supplied as a part of $name
 			$this->CheckFolder($path);
 
-			do {
+			while (file_exists($path . '/' . $new_name) || in_array($path . '/' . $new_name, $forbidden_names)) {
 				if ( preg_match('/(.*)_([0-9]*)(' . preg_quote($ext, '/') . ')/', $new_name, $regs) ) {
 					$new_name = $regs[1] . '_' . ((int)$regs[2] + 1) . $regs[3];
 				}
@@ -450,8 +450,7 @@
 				}
 
 				$original_checked = true;
-				$res = file_exists($path . '/' . $new_name) || in_array($path . '/' . $new_name, $forbidden_names);
-			} while ($res !== false);
+			}
 
 			if ( $parts['dirname'] != '.' ) {
 				$new_name = $parts['dirname'] . '/' . $new_name;