bk commit into 5.1 tree (tnurnberg:1.2493)

[Tatjana A Nuernberg]

Below is the list of changes that have just been committed into a local
5.1 repository of tnurnberg. When tnurnberg does a push these changes will
be propagated to the main repository and, within 24 hours after the
push, to the public repository.
For information on how to access the public repository
see http://dev.mysql.com/doc/mysql/en/installing-source-tree.html

ChangeSet@stripped, 2007-04-30 10:37:50+02:00, tnurnberg@stripped +3 -0
  Merge tnurnberg@stripped:/home/bk/mysql-5.1-maint
  into  mysql.com:/home/tnurnberg/27293/51-27293
  MERGE: 1.2457.3.1

  client/mysqldump.c@stripped, 2007-04-30 10:37:10+02:00, tnurnberg@stripped +0 -0
    Auto merged
    MERGE: 1.268.3.1

  mysql-test/r/mysqldump.result@stripped, 2007-04-30 10:37:10+02:00, tnurnberg@stripped +0 -0
    Auto merged
    MERGE: 1.137.2.1

  mysql-test/t/mysqldump.test@stripped, 2007-04-30 10:37:34+02:00, tnurnberg@stripped +0 -36
    manual merge
    MERGE: 1.130.3.1

# This is a BitKeeper patch.  What follows are the unified diffs for the
# set of deltas contained in the patch.  The rest of the patch, the part
# that BitKeeper cares about, is below these diffs.
# User:	tnurnberg
# Host:	blasphemy.mysql.com
# Root:	/home/tnurnberg/27293/51-27293/RESYNC

--- 1.275/client/mysqldump.c	2007-04-20 23:31:06 +02:00
+++ 1.276/client/mysqldump.c	2007-04-30 10:37:10 +02:00
@@ -1640,9 +1640,16 @@ static uint dump_routines_for_db(char *d
             if the user has EXECUTE privilege he see routine names, but NOT the
             routine body of other routines that are not the creator of!
           */
-          DBUG_PRINT("info",("length of body for %s row[2] '%s' is %ld",
-                             routine_name, row[2], (long) strlen(row[2])));
-          if (strlen(row[2]))
+          DBUG_PRINT("info",("length of body for %s row[2] '%s' is %d",
+                             routine_name, row[2] ? row[2] : "(null)",
+                             row[2] ? (int) strlen(row[2]) : 0));
+          if (row[2] == NULL)
+          {
+            fprintf(sql_file, "\n-- insufficient privileges to %s\n", query_buff);
+            fprintf(sql_file, "-- does %s have permissions on mysql.proc?\n\n", current_user);
+            safe_exit(EX_MYSQLERR);
+          }
+          else if (strlen(row[2]))
           {
             char *query_str= NULL;
             char *definer_begin;
@@ -1692,7 +1699,7 @@ static uint dump_routines_for_db(char *d
             /*
               we need to change sql_mode only for the CREATE
               PROCEDURE/FUNCTION otherwise we may need to re-quote routine_name
-            */;
+            */
             fprintf(sql_file, "/*!50003 SET SESSION SQL_MODE=\"%s\"*/;;\n",
                     row[1] /* sql_mode */);
             fprintf(sql_file, "/*!50003 %s */;;\n",

--- 1.145/mysql-test/r/mysqldump.result	2007-04-02 10:54:26 +02:00
+++ 1.146/mysql-test/r/mysqldump.result	2007-04-30 10:37:10 +02:00
@@ -3204,6 +3204,33 @@ INSERT INTO t1 VALUES(1,0xff00fef0);
 </database>
 </mysqldump>
 DROP TABLE t1;
+# 
+# Bug 27293: mysqldump crashes when dumping routines
+# defined by a different user
+#
+# Bug #22761: mysqldump reports no errors when using
+# --routines without mysql.proc privileges
+#
+create database mysqldump_test_db;
+grant all privileges on mysqldump_test_db.*  to user1;
+grant all privileges on mysqldump_test_db.*  to user2;
+create procedure mysqldump_test_db.sp1() select 'hello';
+DELIMITER ;;
+
+-- insufficient privileges to SHOW CREATE PROCEDURE `sp1`
+-- does user2 have permissions on mysql.proc?
+
+DELIMITER ;
+DELIMITER ;;
+/*!50003 SET SESSION SQL_MODE=""*/;;
+/*!50003 CREATE*/ /*!50020 DEFINER=`user1`@`%`*/ /*!50003 PROCEDURE `sp1`()
+select 'hello' */;;
+/*!50003 SET SESSION SQL_MODE=@OLD_SQL_MODE*/;;
+DELIMITER ;
+drop procedure sp1;
+drop user user1;
+drop user user2;
+drop database mysqldump_test_db;
 #
 # Bug#26346:  	stack + buffer overrun in mysqldump
 #