BUILD : 1.7.18 (1220) BUGS : 657 NOTES : Fixed various OperServ commands which did not respect a disabled OSOpersOnly

git-svn-id: svn://svn.anope.org/anope/trunk@1220 31f1291d-b8d6-0310-a050-a5561fc1590b


git-svn-id: http://anope.svn.sourceforge.net/svnroot/anope/trunk@939 5417fbe8-f217-4b02-8779-1006273d7864

Changes

@@ -5,6 +5,7 @@ Anope Version  S V N
 01/02 F Error reporting on LIST commands accepting ranges.                [#622]
 01/02 F ChanServ LIST now searches for '#chan' if only 'chan' is given.   [#622]
 01/06 F Missing backtick in SQL query for saving news items.              [ #00]
+01/06 F Some OperServ commands not respecting OSOpersOnly when disabled.  [#657]
 
 Anope Version 1.7.18
 --------------------

src/core/os_admin.c

@@ -35,7 +35,7 @@ int AnopeInit(int argc, char **argv)
     moduleAddVersion("$Id$");
     moduleSetType(CORE);
 
-    c = createCommand("ADMIN", do_admin, is_oper, OPER_HELP_ADMIN, -1, -1,
+    c = createCommand("ADMIN", do_admin, NULL, OPER_HELP_ADMIN, -1, -1,
                       -1, -1);
     moduleAddCommand(OPERSERV, c, MOD_UNIQUE);
 
@@ -60,7 +60,7 @@ void AnopeFini(void)
  **/
 void myOperServHelp(User * u)
 {
-    if (is_oper(u)) {
+    if (is_oper(u) || is_services_root(u)) {
         notice_lang(s_OperServ, u, OPER_HELP_CMD_ADMIN);
     }
 }
@@ -172,7 +172,12 @@ int do_admin(User * u)
             notice_lang(s_OperServ, u, READ_ONLY_MODE);
     } else if (!stricmp(cmd, "LIST")) {
         int sent_header = 0;
-
+		
+		if (!is_oper(u)) {
+			notice_lang(s_OperServ, u, PERMISSION_DENIED);
+			return MOD_CONT;
+		}
+		
         if (servadmins.count == 0) {
             notice_lang(s_OperServ, u, OPER_ADMIN_LIST_EMPTY);
             return MOD_CONT;

src/core/os_help.c

@@ -30,7 +30,7 @@ int AnopeInit(int argc, char **argv)
     moduleAddAuthor("Anope");
     moduleAddVersion("$Id$");
     moduleSetType(CORE);
-    c = createCommand("HELP", do_help, is_oper, -1, -1, -1, -1, -1);
+    c = createCommand("HELP", do_help, NULL, -1, -1, -1, -1, -1);
     moduleAddCommand(OPERSERV, c, MOD_UNIQUE);
 
     return MOD_CONT;

src/core/os_modinfo.c

@@ -33,7 +33,7 @@ int AnopeInit(int argc, char **argv)
     moduleAddAuthor("Anope");
     moduleAddVersion("$Id$");
     moduleSetType(CORE);
-    c = createCommand("MODINFO", do_modinfo, is_oper, -1, -1, -1, -1,
+    c = createCommand("MODINFO", do_modinfo, NULL, -1, -1, -1, -1,
                       OPER_HELP_MODINFO);
     moduleAddCommand(OPERSERV, c, MOD_UNIQUE);
 
@@ -57,9 +57,7 @@ void AnopeFini(void)
  **/
 void myOperServHelp(User * u)
 {
-    if (is_oper(u)) {
-        notice_lang(s_OperServ, u, OPER_HELP_CMD_MODINFO);
-    }
+    notice_lang(s_OperServ, u, OPER_HELP_CMD_MODINFO);
 }
 
 /**

src/core/os_modlist.c

@@ -32,7 +32,7 @@ int AnopeInit(int argc, char **argv)
     moduleAddVersion("$Id$");
     moduleSetType(CORE);
 
-    c = createCommand("MODLIST", do_modlist, is_oper, -1, -1, -1, -1,
+    c = createCommand("MODLIST", do_modlist, NULL, -1, -1, -1, -1,
                       OPER_HELP_MODLIST);
     moduleAddCommand(OPERSERV, c, MOD_UNIQUE);
 
@@ -56,9 +56,7 @@ void AnopeFini(void)
  **/
 void myOperServHelp(User * u)
 {
-    if (is_oper(u)) {
-        notice_lang(s_OperServ, u, OPER_HELP_CMD_MODLIST);
-    }
+    notice_lang(s_OperServ, u, OPER_HELP_CMD_MODLIST);
 }
 
 /**

src/core/os_oper.c

@@ -34,7 +34,7 @@ int AnopeInit(int argc, char **argv)
     moduleAddAuthor("Anope");
     moduleAddVersion("$Id$");
     moduleSetType(CORE);
-    c = createCommand("OPER", do_oper, is_oper, OPER_HELP_OPER, -1, -1, -1,
+    c = createCommand("OPER", do_oper, NULL, OPER_HELP_OPER, -1, -1, -1,
                       -1);
     moduleAddCommand(OPERSERV, c, MOD_UNIQUE);
 
@@ -58,7 +58,7 @@ void AnopeFini(void)
  **/
 void myOperServHelp(User * u)
 {
-    if (is_oper(u)) {
+    if (is_oper(u) || is_services_admin(u)) {
         notice_lang(s_OperServ, u, OPER_HELP_CMD_OPER);
     }
 }
@@ -165,6 +165,11 @@ int do_oper(User * u)
     } else if (!stricmp(cmd, "LIST")) {
         int sent_header = 0;
 
+		if (!is_oper(u)) {
+			notice_lang(s_OperServ, u, PERMISSION_DENIED);
+			return MOD_CONT;
+		}
+		
         if (servopers.count == 0) {
             notice_lang(s_OperServ, u, OPER_OPER_LIST_EMPTY);
             return MOD_CONT;

src/core/os_staff.c

@@ -35,7 +35,7 @@ int AnopeInit(int argc, char **argv)
     moduleAddVersion("$Id$");
     moduleSetType(CORE);
 
-    c = createCommand("STAFF", do_staff, is_oper, OPER_HELP_STAFF, -1, -1,
+    c = createCommand("STAFF", do_staff, NULL, OPER_HELP_STAFF, -1, -1,
                       -1, -1);
     moduleAddCommand(OPERSERV, c, MOD_UNIQUE);
 
@@ -59,9 +59,7 @@ void AnopeFini(void)
  **/
 void myOperServHelp(User * u)
 {
-    if (is_oper(u)) {
-        notice_lang(s_OperServ, u, OPER_HELP_CMD_STAFF);
-    }
+    notice_lang(s_OperServ, u, OPER_HELP_CMD_STAFF);
 }
 
 /**

version.log

@@ -9,10 +9,14 @@ VERSION_MAJOR="1"
 VERSION_MINOR="7"
 VERSION_PATCH="18"
 VERSION_EXTRA="-svn"
-VERSION_BUILD="1219"
+VERSION_BUILD="1220"
 
 # $Log$
 #
+# BUILD : 1.7.18 (1220)
+# BUGS  : 657
+# NOTES : Fixed various OperServ commands which did not respect a disabled OSOpersOnly
+#
 # BUILD : 1.7.18 (1219)
 # BUGS  : 
 # NOTES : Fixed a missing backtick in the INSERT query for newsitems