Index: branches/5.1.x/core/units/admin/admin_events_handler.php
===================================================================
diff -u -N -r13433 -r13569
--- branches/5.1.x/core/units/admin/admin_events_handler.php	(.../admin_events_handler.php)	(revision 13433)
+++ branches/5.1.x/core/units/admin/admin_events_handler.php	(.../admin_events_handler.php)	(revision 13569)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: admin_events_handler.php 13433 2010-04-27 10:23:52Z alex $
+* @version	$Id: admin_events_handler.php 13569 2010-05-18 15:25:01Z alex $
 * @package	In-Portal
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -681,7 +681,18 @@
 	 */
 	function OnBackup(&$event)
 	{
+		$backup_path = $this->Application->ConfigValue('Backup_Path');
 
+		$file_helper =& $this->Application->recallObject('FileHelper');
+		/* @var $file_helper FileHelper */
+
+		if (!$file_helper->CheckFolder($backup_path) || !is_writable($backup_path)) {
+			$event->status = erFAIL;
+
+			$this->Application->SetVar('error_msg', $this->Application->Phrase('la_Text_backup_access'));
+			return ;
+		}
+
 		$a_tables = $this->Conn->GetCol('SHOW TABLES'); //  array_keys($tables);
 		$TableNames = Array();
 		for($x=0;$x<count($a_tables);$x++)
@@ -693,15 +704,13 @@
 		  	  	}
 		  	}
 		}
-//			echo "<pre>"; print_r($TableNames); echo "</pre>";
-//			exit;
 
 		$backupProgress = Array (
 			'table_num' => 0,
 			'table_names' => $TableNames,
 			'table_count' => count($TableNames),
 			'record_count' => 0,
-			'file_name' => $this->Application->ConfigValue('Backup_Path')."/dump".adodb_mktime().".txt",
+			'file_name' => $backup_path."/dump".adodb_mktime().".txt",
 		);
 		$this->Application->RemoveVar('adm.backupcomplete_filename');
 		$this->Application->RemoveVar('adm.backupcomplete_filesize');