BUILD : 1.7.8 (593) BUGS : 307 NOTES : Fixed admin/owner mode handling

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


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

2 files changed, 33 insertions, 14 deletions

diff --git a/src/chanserv.c b/src/chanserv.c
index 3e2a4a1cb..521345057 100644
--- a/src/chanserv.c
+++ b/src/chanserv.c
@@ -1413,6 +1413,7 @@ int check_valid_admin(User * user, Channel * chan, int servermode)
 
 int check_valid_op(User * user, Channel * chan, int servermode)
 {
+    char *tmp;
     if (!chan || !chan->ci)
         return 1;
 
@@ -1425,25 +1426,29 @@ int check_valid_op(User * user, Channel * chan, int servermode)
         if (ircd->halfop) {
             if (ircd->owner && ircd->protect) {
                 if (check_access(user, chan->ci, CA_AUTOHALFOP)) {
+                    tmp = stripModePrefix(ircd->ownerunset);
                     anope_cmd_mode(whosends(chan->ci), chan->name,
-                                   "-%so%s %s %s %s", ++ircd->adminunset,
-                                   ++ircd->ownerunset, user->nick,
+                                   "%so%s %s %s %s", ircd->adminunset,
+                                   tmp, user->nick,
                                    user->nick, user->nick);
+                    free(tmp);
                 } else {
+                    tmp = stripModePrefix(ircd->ownerunset);
                     anope_cmd_mode(whosends(chan->ci), chan->name,
-                                   "-%sho%s %s %s %s %s",
-                                   ++ircd->adminunset, ++ircd->ownerunset,
+                                   "%sho%s %s %s %s %s",
+                                   ircd->adminunset, tmp,
                                    user->nick, user->nick, user->nick,
                                    user->nick);
+                    free(tmp);
                 }
             } else if (!ircd->owner && ircd->protect) {
                 if (check_access(user, chan->ci, CA_AUTOHALFOP)) {
                     anope_cmd_mode(whosends(chan->ci), chan->name,
-                                   "-%so %s %s", ++ircd->adminunset,
+                                   "%so %s %s", ircd->adminunset,
                                    user->nick, user->nick);
                 } else {
                     anope_cmd_mode(whosends(chan->ci), chan->name,
-                                   "-%soh %s %s %s", ++ircd->adminunset,
+                                   "%soh %s %s %s", ircd->adminunset,
                                    user->nick, user->nick, user->nick);
                 }
             } else {
@@ -1465,10 +1470,12 @@ int check_valid_op(User * user, Channel * chan, int servermode)
     if (check_access(user, chan->ci, CA_AUTODEOP)) {
         if (ircd->halfop) {
             if (ircd->owner && ircd->protect) {
+                tmp = stripModePrefix(ircd->ownerunset);
                 anope_cmd_mode(whosends(chan->ci), chan->name,
-                               "-%sho%s %s %s %s %s", ++ircd->adminunset,
-                               ++ircd->ownerunset, user->nick, user->nick,
+                               "%sho%s %s %s %s %s", ircd->adminunset,
+                               tmp, user->nick, user->nick,
                                user->nick, user->nick);
+                free(tmp);
             } else {
                 anope_cmd_mode(whosends(chan->ci), chan->name, "-ho %s %s",
                                user->nick, user->nick);
@@ -6093,13 +6100,13 @@ static int do_clear(User * u)
                         if (!chan_has_user_status(c, cu->user, CUS_OWNER)) {
                             continue;
                         } else {
-                            snprintf(tmp, BUFSIZE, "-%so",
-                                     ++ircd->ownerunset);
+                            snprintf(tmp, BUFSIZE, "%so",
+                                     ircd->ownerunset);
                             av[1] = sstrdup(tmp);
 
                         }
                     } else {
-                        snprintf(tmp, BUFSIZE, "-%so", ++ircd->adminunset);
+                        snprintf(tmp, BUFSIZE, "%so", ircd->adminunset);
                         av[1] = sstrdup(tmp);
                     }
                 } else {
@@ -6120,12 +6127,12 @@ static int do_clear(User * u)
                         if (!chan_has_user_status(c, cu->user, CUS_OWNER)) {
                             continue;
                         } else {
-                            snprintf(tmp, BUFSIZE, "-%so",
-                                     ++ircd->ownerunset);
+                            snprintf(tmp, BUFSIZE, "%so",
+                                     ircd->ownerunset);
                             av[1] = sstrdup(tmp);
                         }
                     } else {
-                        snprintf(tmp, BUFSIZE, "-%so", ++ircd->adminunset);
+                        snprintf(tmp, BUFSIZE, "%so", ircd->adminunset);
                         av[1] = sstrdup(tmp);
                     }
                 } else {
diff --git a/src/misc.c b/src/misc.c
index 5e5038252..3df45aab5 100644
--- a/src/misc.c
+++ b/src/misc.c
@@ -1135,4 +1135,16 @@ char *str_signed(unsigned char *str)
 	return nstr;
 }
 
+/**
+ *  Strip the mode prefix from the given string.
+ *  Useful for using the modes stored in things like ircd->ownerset etc.. 
+ **/
+
+char *stripModePrefix(const char *str) {
+    if(str) {
+        return strdup(str+1);
+    }
+    return NULL;
+}
+
 /* EOF */