From: Date: May 26 2005 10:50am
Subject: bk commit into 5.0 tree (msvensson:1.1839) BUG#9993
List-Archive: http://lists.mysql.com/internals/25295
X-Bug: 9993
Message-Id: <20050526085041.19B4F25DF7C@blaudden.homeip.net>

Below is the list of changes that have just been committed into a local
5.0 repository of msvensson. When msvensson 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
  1.1839 05/05/26 10:50:36 msvensson@neptunus.(none) +4 -0
  BUG#9993 2 unexpected warnings when dropping a routine and --skip-grant-tables
   - Turn off automatic privilege handling for stored procedures when --skip-grant-tables
   - Renamed view_skip_grants to skip_grants and added test cases for this

  sql/mysqld.cc
    1.454 05/05/26 10:49:30 msvensson@neptunus.(none) +3 -0
    Turn off automatic privileges for stored proceduress if --skip-grant-tables are used.

  mysql-test/t/skip_grants.test
    1.3 05/05/26 10:49:30 msvensson@neptunus.(none) +9 -1
    Added test for create and drop procedure with --skip-grant-tables

  mysql-test/r/skip_grants.result
    1.3 05/05/26 10:49:30 msvensson@neptunus.(none) +4 -0
    Added test for create and drop procedure with --skip-grant-tables

  mysql-test/t/skip_grants-master.opt
    1.2 05/05/26 10:40:47 msvensson@neptunus.(none) +0 -0
    Rename: mysql-test/t/view_skip_grants-master.opt -> mysql-test/t/skip_grants-master.opt

  mysql-test/r/skip_grants.result
    1.2 05/05/26 10:40:23 msvensson@neptunus.(none) +0 -0
    Rename: mysql-test/r/view_skip_grants.result -> mysql-test/r/skip_grants.result

  mysql-test/t/skip_grants.test
    1.2 05/05/26 10:40:04 msvensson@neptunus.(none) +0 -0
    Rename: mysql-test/t/view_skip_grants.test -> mysql-test/t/skip_grants.test

# 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:	msvensson
# Host:	neptunus.(none)
# Root:	/home/msvensson/mysql/bug9993

--- 1.453/sql/mysqld.cc	2005-05-16 08:12:32 +02:00
+++ 1.454/sql/mysqld.cc	2005-05-26 10:49:30 +02:00
@@ -6325,6 +6325,9 @@
   case (int) OPT_SKIP_LOCK:
     opt_external_locking=0;
     break;
+  case (int) OPT_SKIP_GRANT:
+    sp_automatic_privileges=0;
+    break;
   case (int) OPT_SKIP_HOST_CACHE:
     opt_specialflag|= SPECIAL_NO_HOST_CACHE;
     break;

--- 1.1/mysql-test/r/view_skip_grants.result	2004-07-20 00:00:59 +02:00
+++ 1.3/mysql-test/r/skip_grants.result	2005-05-26 10:49:30 +02:00
@@ -1,6 +1,10 @@
 drop table if exists t1,v1;
 drop view if exists t1,v1;
+drop procedure if exists f1;
 use test;
 create table t1 (field1 INT);
 CREATE VIEW v1 AS SELECT field1 FROM t1;
 drop view v1;
+drop table t1;
+create procedure f1() select 1;
+drop procedure f1;

--- 1.1/mysql-test/t/view_skip_grants.test	2004-07-20 00:00:59 +02:00
+++ 1.3/mysql-test/t/skip_grants.test	2005-05-26 10:49:30 +02:00
@@ -1,6 +1,7 @@
 --disable_warnings
 drop table if exists t1,v1;
 drop view if exists t1,v1;
+drop procedure if exists f1;
 --enable_warnings
 use test;
 
@@ -11,4 +12,11 @@
 CREATE VIEW v1 AS SELECT field1 FROM t1;
 
 drop view v1;
-drop table t1
+drop table t1;
+
+#
+# Test that we can create and drop procedure without warnings
+# see bug#9993
+#
+create procedure f1() select 1;
+drop procedure f1;