]> jfr.im git - irc/quakenet/newserv.git/blobdiff - newsearch/newsearch.c
help changes
[irc/quakenet/newserv.git] / newsearch / newsearch.c
index 6726213ca5eaef2657cc805d3ec0622b76b0574f..68785cfbe06906412366d31e546d7e3e426d48ab 100644 (file)
@@ -1,7 +1,6 @@
 #include <stdio.h>
 #include <stdarg.h>
 #include <string.h>
-#include "newsearch.h"
 
 #include "../irc/irc_config.h"
 #include "../lib/irc_string.h"
 #include "../lib/version.h"
 #include "../lib/stringbuf.h"
 #include "../lib/strlfunc.h"
+#include "../lib/array.h"
+#include "newsearch.h"
 
 MODULE_VERSION("");
 
-CommandTree *searchTree;
-CommandTree *chanOutputTree;
-CommandTree *nickOutputTree;
-CommandTree *userOutputTree;
+CommandTree *searchCmdTree;
+searchList *globalterms = NULL;
 
 int do_nicksearch(void *source, int cargc, char **cargv);
 int do_chansearch(void *source, int cargc, char **cargv);
@@ -32,139 +31,314 @@ UserDisplayFunc defaultuserfn = printuser;
 NickDisplayFunc defaultnickfn = printnick;
 ChanDisplayFunc defaultchanfn = printchannel;
 
-void registersearchterm(char *term, parseFunc parsefunc);
-void deregistersearchterm(char *term, parseFunc parsefunc);
+searchCmd *reg_nicksearch, *reg_chansearch, *reg_usersearch;
+void displaycommandhelp(nick *, Command *);
 
-void regchandisp(const char *name, ChanDisplayFunc handler) {
-  addcommandtotree(chanOutputTree, name, 0, 0, (CommandHandler)handler);
-}
+searchCmd *registersearchcommand(char *name, int level, CommandHandler cmd, void *defaultdisplayfunc) {
+  searchCmd *acmd;
+  searchList *sl;
 
-void unregchandisp(const char *name, ChanDisplayFunc handler) {
-  deletecommandfromtree(chanOutputTree, name, (CommandHandler)handler);
-}
+  registercontrolhelpfunccmd(name, NO_OPER,4, cmd, &displaycommandhelp);  
+  acmd=(struct searchCmd *)malloc(sizeof(struct searchCmd));
+  if (!acmd) {
+    Error("newsearch", ERR_ERROR, "malloc failed: registersearchcommand");
+    return NULL;
+  }
+  acmd->handler = cmd;
+
+  acmd->name = getsstring( name, NSMAX_COMMAND_LEN); 
+  acmd->outputtree = newcommandtree();
+  acmd->searchtree = newcommandtree();
+
+  addcommandtotree(searchCmdTree, name, 0, 0, (CommandHandler)acmd);
+
+  for (sl=globalterms; sl; sl=sl->next) {
+    registersearchterm( acmd, sl->name->content, sl->cmd, 0, sl->help);
+  }
 
-void regnickdisp(const char *name, NickDisplayFunc handler) {
-  addcommandtotree(nickOutputTree, name, 0, 0, (CommandHandler)handler);
+  return acmd;
 }
 
-void unregnickdisp(const char *name, NickDisplayFunc handler) {
-  deletecommandfromtree(nickOutputTree, name, (CommandHandler)handler);
+void deregistersearchcommand(searchCmd *scmd) {
+  deregistercontrolcmd(scmd->name->content, (CommandHandler)scmd->handler);
+  destroycommandtree(scmd->outputtree);
+  destroycommandtree(scmd->searchtree);
+  freesstring(scmd->name);
+  free(scmd);
 }
 
-void reguserdisp(const char *name, UserDisplayFunc handler) {
-  addcommandtotree(userOutputTree, name, 0, 0, (CommandHandler)handler);
+void displaycommandhelp(nick *np, Command *cmd) {
+  int i,n,j,m;
+  Command *cmdlist[100], *acmdlist[100];
+  searchCmd *acmd;
+
+  n=getcommandlist(searchCmdTree,cmdlist,100);
+  for(i=0;i<n;i++) {
+    /* note: we may want to only deregister a command if we've already registered it, for now, try de-registering new commands anyway */
+    if ( ((searchCmd *)cmdlist[i]->handler)->handler != cmd->handler )
+      continue;
+    acmd = ((searchCmd *)(cmdlist[i]->handler)); 
+
+    controlreply(np, "Usage: [flags] <criteria>\n");
+    controlreply(np, "Flags:\n");
+    controlreply(np, " -l int    : Limit number of rows of results\n");
+    controlreply(np, " -d string : a valid output format for the results\n"); 
+    controlreply(np, " -s subset : ipmask subset of network to search (only for node search)\n");
+    controlreply(np, " \n");
+    controlreply(np, "Available Output Formats:\n");
+  
+    m=getcommandlist(acmd->outputtree,acmdlist,100);
+    for(j=0;j<m;j++) {
+      if ( controlpermitted( acmdlist[j]->level, np) ) {
+        char *help=(char *)acmdlist[j]->ext;
+        if ( help && help[0] != '\0')
+          controlreply(np, "%-10s: %s\n", acmdlist[j]->command->content, help);
+        else 
+          controlreply(np, "%s\n", acmdlist[j]->command->content ); 
+      }
+    }
+
+    controlreply(np, " \n");
+    controlreply(np, "Available Global Commands and Operators:\n" );
+    m=getcommandlist(acmd->searchtree,acmdlist,100);
+    for(j=0;j<m;j++) {
+      if ( acmdlist[j]->maxparams) {
+        char *help=(char *)acmdlist[j]->ext;
+        if ( help && help[0] != '\0')
+          controlreply(np, "%-10s: %s\n", acmdlist[j]->command->content, help );
+        else
+          controlreply(np, "%s\n", acmdlist[j]->command->content );
+      }
+    }
+
+    controlreply(np, " \n");
+    controlreply(np, "Available Commands and Operators for %s:\n", acmd->name->content);
+
+    m=getcommandlist(acmd->searchtree,acmdlist,100);
+    for(j=0;j<m;j++) {
+      if ( !acmdlist[j]->maxparams && controlpermitted( acmdlist[j]->level, np) ) {
+        char *help=(char *)acmdlist[j]->ext;
+        if ( help && help[0] != '\0')
+          controlreply(np, "%-10s: %s\n", acmdlist[j]->command->content, help );
+        else
+          controlreply(np, "%s\n", acmdlist[j]->command->content );
+      }
+    }
+  }
 }
 
-void unreguserdisp(const char *name, UserDisplayFunc handler) {
-  deletecommandfromtree(userOutputTree, name, (CommandHandler)handler);
+void regdisp( searchCmd *cmd, const char *name, void *handler, int level, char *help) {
+  addcommandexttotree(cmd->outputtree, name, level, 0, (CommandHandler) handler, (char *)help);
+} 
+
+void unregdisp( searchCmd *cmd, const char *name, void *handler ) {
+  deletecommandfromtree(cmd->outputtree, name, (CommandHandler) handler);
 }
 
+void *findcommandinlist( searchList *sl, char *name){
+  while (sl) {
+    if(strcmp(sl->name->content,name) == 0 ) {
+      return sl;
+    }
+    sl=sl->next;
+  }
+  return NULL;
+} 
+
 const char *parseError;
 /* used for *_free functions that need to warn users of certain things
    i.e. hitting too many users in a (kill) or (gline) */
 nick *senderNSExtern;
 
 void _init() {
-  searchTree=newcommandtree();
-  chanOutputTree=newcommandtree();
-  nickOutputTree=newcommandtree();
-  userOutputTree=newcommandtree();
+  searchCmdTree=newcommandtree();
+
+  reg_nicksearch = (searchCmd *)registersearchcommand("nicksearch",NO_OPER,&do_nicksearch, printnick);
+  reg_chansearch = (searchCmd *)registersearchcommand("chansearch",NO_OPER,&do_chansearch, printchannel);
+  reg_usersearch = (searchCmd *)registersearchcommand("usersearch",NO_OPER,&do_usersearch, printuser);
 
   /* Boolean operations */
-  registersearchterm("and",and_parse);
-  registersearchterm("not",not_parse);
-  registersearchterm("or",or_parse);
+  registerglobalsearchterm("and",and_parse, "usage: (and (X) (X))" );
+  registerglobalsearchterm("not",not_parse, "usage: (not (X))");
+  registerglobalsearchterm("or",or_parse, "usage: (or (X) (X))" );
 
-  registersearchterm("eq",eq_parse);
+  registerglobalsearchterm("eq",eq_parse, "usage: (eq (X) Y)");
 
-  registersearchterm("lt",lt_parse);
-  registersearchterm("gt",gt_parse);
+  registerglobalsearchterm("lt",lt_parse, "usage: (lt (X) int)");
+  registerglobalsearchterm("gt",gt_parse, "usage: (gt (X) int)");
  
   /* String operations */
-  registersearchterm("match",match_parse);
-  registersearchterm("regex",regex_parse);
-  registersearchterm("length",length_parse);
+  registerglobalsearchterm("match",match_parse, "usage: (match (X) string)");
+  registerglobalsearchterm("regex",regex_parse, "usage: (regex (X) string)");
+  registerglobalsearchterm("length",length_parse, "usage: (length string)");
   
   /* Nickname operations */
-  registersearchterm("hostmask",hostmask_parse);
-  registersearchterm("realname",realname_parse);
-  registersearchterm("authname",authname_parse);
-  registersearchterm("authts",authts_parse);
-  registersearchterm("ident",ident_parse);
-  registersearchterm("host",host_parse);
-  registersearchterm("channel",channel_parse);
-  registersearchterm("timestamp",timestamp_parse);
-  registersearchterm("country",country_parse);
-  registersearchterm("ip",ip_parse);
-  registersearchterm("channels",channels_parse);
-  registersearchterm("server",server_parse);
-  registersearchterm("authid",authid_parse);
+  registersearchterm(reg_nicksearch, "hostmask",hostmask_parse, 0, "The user's nick!user@host; \"hostmask real\" returns nick!user@host\rreal");     /* nick only */
+  registersearchterm(reg_nicksearch, "realname",realname_parse, 0, "User's current realname");     /* nick only */
+  registersearchterm(reg_nicksearch, "authname",authname_parse, 0, "User's current authname or false");     /* nick only */
+  registersearchterm(reg_nicksearch, "authts",authts_parse, 0, "User's Auth timestamp");         /* nick only */
+  registersearchterm(reg_nicksearch, "ident",ident_parse, 0, "User's current ident");           /* nick only */
+  registersearchterm(reg_nicksearch, "host",host_parse, 0, "User's host, allow \"host real\" to match real host");             /* nick only */
+  registersearchterm(reg_nicksearch, "channel",channel_parse, 0, "Valid Channel Name to match users against");       /* nick only */
+  registersearchterm(reg_nicksearch, "timestamp",timestamp_parse, 0, "User's Timestamp");   /* nick only */
+  registersearchterm(reg_nicksearch, "country",country_parse, 0, "2 letter country code (data source is geoip)");       /* nick only */
+  registersearchterm(reg_nicksearch, "ip",ip_parse, 0, "User's IP - ipv4 or ipv6 format as appropriate. Note: not 6to4");                 /* nick only */
+  registersearchterm(reg_nicksearch, "channels",channels_parse, 0, "Channel Count");     /* nick only */
+  registersearchterm(reg_nicksearch, "server",server_parse, 0, "Server Name. Either (server string) or (match (server) string)");         /* nick only */
+  registersearchterm(reg_nicksearch, "authid",authid_parse, 0, "User's Auth ID");         /* nick only */
 
   /* Channel operations */
-  registersearchterm("exists",exists_parse);
-  registersearchterm("services",services_parse);
-  registersearchterm("size",size_parse);
-  registersearchterm("name",name_parse);
-  registersearchterm("topic",topic_parse);
-  registersearchterm("oppct",oppct_parse);
-  registersearchterm("uniquehostpct",hostpct_parse);
-  registersearchterm("authedpct",authedpct_parse);
-  registersearchterm("kick",kick_parse);
+  registersearchterm(reg_chansearch, "exists",exists_parse, 0, "Returns if channel exists on network. Note: newserv may store data on empty channels");         /* channel only */
+  registersearchterm(reg_chansearch, "services",services_parse, 0, "");     /* channel only */
+  registersearchterm(reg_chansearch, "size",size_parse, 0, "Channel user count");             /* channel only */
+  registersearchterm(reg_chansearch, "name",name_parse, 0, "Channel Name");             /* channel only */
+  registersearchterm(reg_chansearch, "topic",topic_parse, 0, "Channel topic");           /* channel only */
+  registersearchterm(reg_chansearch, "oppct",oppct_parse, 0, "Percentage Opped");           /* channel only */
+  registersearchterm(reg_chansearch, "uniquehostpct",hostpct_parse, 0, "uniquehost percent"); /* channel only */
+  registersearchterm(reg_chansearch, "authedpct",authedpct_parse, 0, "Percentage of authed users");   /* channel only */
+  registersearchterm(reg_chansearch, "kick",kick_parse, 0, "KICK users channels in newsearch result. Note: evaluation order");             /* channel only */
 
   /* Nickname / channel operations */
-  registersearchterm("modes",modes_parse);
-  registersearchterm("nick",nick_parse);
+  registersearchterm(reg_chansearch, "modes",modes_parse, 0, "User Modes");
+  registersearchterm(reg_nicksearch, "modes",modes_parse, 0, "Channel Modes");
+  registersearchterm(reg_chansearch, "nick",nick_parse, 0, "Nickname");
+  registersearchterm(reg_nicksearch, "nick",nick_parse, 0, "Nickname");
 
   /* Kill / gline parameters */
-  registersearchterm("kill",kill_parse);
-  registersearchterm("gline",gline_parse);
+  registersearchterm(reg_chansearch,"kill",kill_parse, 0, "KILL users in newsearch result. Note: evaluation order");
+  registersearchterm(reg_chansearch,"gline",gline_parse, 0, "GLINE users in newsearch result. Note: evaluation order");
+  registersearchterm(reg_nicksearch,"kill",kill_parse, 0, "KILL users in newsearch result. Note: evaluation order");
+  registersearchterm(reg_nicksearch,"gline",gline_parse, 0, "GLINE users in newsearch result. Note: evaluation order");
 
   /* Iteration functionality */
-  registersearchterm("any",any_parse);
-  registersearchterm("all",all_parse);
-  registersearchterm("var",var_parse);
+  registerglobalsearchterm("any",any_parse, "usage: any (generatorfn x) (fn ... (var x) ...)");
+  registerglobalsearchterm("all",all_parse, "usage: all (generatorfn x) (fn ... (var x) ...)");
+  registerglobalsearchterm("var",var_parse, "usage: var variable");
   
   /* Iterable functions */
-  registersearchterm("channeliter",channeliter_parse);
+  registersearchterm(reg_nicksearch, "channeliter",channeliter_parse, 0, "Channel Iterable function - usage: (any (channeliter x) (match (var x) #twilight*))");         /* nick only */
   
   /* Notice functionality */
-  registersearchterm("notice",notice_parse);
-  
+  registersearchterm(reg_chansearch,"notice",notice_parse, 0, "NOTICE users in newsearch result. Note: evaluation order");
+  registersearchterm(reg_nicksearch,"notice",notice_parse, 0, "NOTICE users in newsearch result. Note: evaluation order");
   /* Nick output filters */
-  regnickdisp("default",printnick);
-  regnickdisp("channels",printnick_channels);
+  regdisp(reg_nicksearch,"default",printnick, 0, "");
+  regdisp(reg_nicksearch,"channels",printnick_channels, 0, "include channels in output");
     
   /* Channel output filters */
-  regchandisp("default",printchannel);
-  regchandisp("topic",printchannel_topic);
-  regchandisp("services",printchannel_services);
+  regdisp(reg_chansearch,"default",printchannel, 0, "");
+  regdisp(reg_chansearch,"topic",printchannel_topic, 0, "display channel topics");
+  regdisp(reg_chansearch,"services",printchannel_services, 0, "display services on channels");
 
   /* Nick output filters */
-  reguserdisp("default",printuser);
-    
-  registercontrolhelpcmd("nicksearch",NO_OPER,4,do_nicksearch, "Usage: nicksearch <criteria>\nSearches for nicknames with the given criteria.");
-  registercontrolhelpcmd("chansearch",NO_OPER,4,do_chansearch, "Usage: chansearch <criteria>\nSearches for channels with the given criteria.");
-  registercontrolhelpcmd("usersearch",NO_OPER,4,do_usersearch, "Usage: usersearch <criteria>\nSearches for users with the given criteria.");
+  regdisp(reg_usersearch,"default",printuser, 0, "");
 }
 
 void _fini() {
-  destroycommandtree(searchTree);
-  destroycommandtree(chanOutputTree);
-  destroycommandtree(nickOutputTree);
-  destroycommandtree(userOutputTree);
-  deregistercontrolcmd("nicksearch", do_nicksearch);
-  deregistercontrolcmd("chansearch", do_chansearch);
-  deregistercontrolcmd("usersearch", do_usersearch);
+  searchList *sl, *psl;
+  int i,n;
+  Command *cmdlist[100];
+
+  sl=globalterms;
+  while (sl) {
+    psl = sl;
+    sl=sl->next;
+
+    n=getcommandlist(searchCmdTree,cmdlist,100);
+    for(i=0;i<n;i++) {
+      deregistersearchterm( (searchCmd *)cmdlist[i]->handler, psl->name->content, psl->cmd);
+    }
+
+    freesstring(psl->name);
+    if (psl->help) 
+      free (psl->help);
+    free(psl);
+  }
+
+  deregistersearchcommand( reg_nicksearch );
+  deregistersearchcommand( reg_chansearch );
+  deregistersearchcommand( reg_usersearch );
+  destroycommandtree( searchCmdTree );
+}
+
+void registerglobalsearchterm(char *term, parseFunc parsefunc, char *help) {
+  int i,n;
+  Command *cmdlist[100];
+  searchList *sl = malloc(sizeof(searchList));
+  if (!sl) {
+    Error("newsearch", ERR_ERROR, "malloc failed: registerglobalsearchterm");
+    return;
+  }
+
+  sl->cmd = parsefunc;
+  sl->name = getsstring(term, NSMAX_COMMAND_LEN);
+  sl->next = NULL;
+  if (help) {
+    int len=strlen(help);
+    sl->help=(char *)malloc(len+1);
+    if(!sl->help) {
+      Error("newsearch", ERR_ERROR, "malloc failed: registerglobalsearchterm");
+      return;
+    }
+      strncpy(sl->help, help, len);
+      sl->help[len] = '\0';
+  } else {
+    sl->help=NULL;
+  }
+
+
+  if ( globalterms != NULL ) {
+    sl->next = globalterms;
+  }
+  globalterms = sl;
+
+  n=getcommandlist(searchCmdTree,cmdlist,100);
+  for(i=0;i<n;i++) {
+    /* maxparams is set to 1 to indicate a global term */
+    /* access level is set to 0 for all global terms */
+    addcommandexttotree( ((searchCmd *)cmdlist[i]->handler)->searchtree,term, 0, 1, (CommandHandler) parsefunc, help);
+  }
+}
+
+void deregisterglobalsearchterm(char *term, parseFunc parsefunc) {
+  int i,n;
+  Command *cmdlist[100];
+  searchList *sl, *psl=NULL;
+
+  for (sl=globalterms; sl; sl=sl->next) {
+    if ( strcmp( sl->name->content, term) == 0 ) {
+      break;
+    }
+    psl = sl;
+  }
+
+  if (sl) {
+    if( psl ) {
+      psl->next = sl->next;
+    } 
+
+    n=getcommandlist(searchCmdTree,cmdlist,100);
+    for(i=0;i<n;i++) {
+      deletecommandfromtree( ((searchCmd *)cmdlist[i]->handler)->searchtree, term, (CommandHandler) parsefunc);
+    }
+    freesstring(sl->name);
+    free(sl);
+  }
 }
 
-void registersearchterm(char *term, parseFunc parsefunc) {
-  addcommandtotree(searchTree, term, 0, 0, (CommandHandler) parsefunc);
+void registersearchterm(searchCmd *cmd, char *term, parseFunc parsefunc, int level, char *help) {
+  /* NOTE: global terms are added to the tree elsewhere as we set maxparams to 1 to indicate global */
+  addcommandexttotree(cmd->searchtree, term, level, 0, (CommandHandler) parsefunc, help);
 }
 
-void deregistersearchterm(char *term, parseFunc parsefunc) {
-  deletecommandfromtree(searchTree, term, (CommandHandler) parsefunc);
+void deregistersearchterm(searchCmd *cmd, char *term, parseFunc parsefunc) {
+  /* NOTE: global terms are removed from the tree within deregisterglobalsearchterm */
+  deletecommandfromtree(cmd->searchtree, term, (CommandHandler) parsefunc);
 }
 
+static void controlwallwrapper(int level, char *format, ...) __attribute__ ((format (printf, 2,         3)));
 static void controlwallwrapper(int level, char *format, ...) {
   char buf[1024];
   va_list ap;
@@ -175,10 +349,11 @@ static void controlwallwrapper(int level, char *format, ...) {
   va_end(ap);
 }
 
-static int parseopts(int cargc, char **cargv, int *arg, int *limit, void **display, CommandTree *tree, replyFunc reply, void *sender) {
+int parseopts(int cargc, char **cargv, int *arg, int *limit, void **subset, void **display, CommandTree *sl, replyFunc reply, void *sender) {
   char *ch;
-  struct Command *cmd;
-  
+  Command *cmd;
+  struct irc_in_addr sin; unsigned char bits;
   if (*cargv[0] == '-') {
     /* options */
     (*arg)++;
@@ -187,28 +362,46 @@ static int parseopts(int cargc, char **cargv, int *arg, int *limit, void **displ
       switch(*ch) {
       case 'l':
        if (cargc<*arg) {
-         reply(sender,"Error: -l switch requires an argument");
-         return CMD_USAGE;
+         reply(sender,"Error: -l switch requires an argument (for help, see help <searchcmd>)");
+         return CMD_ERROR;
        }
        *limit=strtoul(cargv[(*arg)++],NULL,10);
        break;
        
       case 'd':
         if (cargc<*arg) {
-          reply(sender,"Error: -d switch requires an argument");
-          return CMD_USAGE;
+          reply(sender,"Error: -d switch requires an argument (for help, see help <searchcmd>)");
+          return CMD_ERROR;
         }
-        cmd=findcommandintree(tree, cargv[*arg], 1);
+        cmd=findcommandintree(sl, cargv[*arg],1);
         if (!cmd) {
-          reply(sender,"Error: unknown output format %s",cargv[*arg]);
-          return CMD_USAGE;
+          reply(sender,"Error: unknown output format %s (for help, see help <searchcmd>)",cargv[*arg]);
+          return CMD_ERROR;
+        }
+        if ( !controlpermitted( cmd->level, sender) ) {
+          reply(sender,"Error: Access Denied for output format %s (for help, see help <searchcmd>)", cargv[*arg]);
+          return CMD_ERROR;
         }
         *display=(void *)cmd->handler;
         (*arg)++;
         break;
+
+      case 's':
+        if (cargc<*arg) {
+          reply(sender,"Error: -s switch requires an argument (for help, see help <searchcmd>)");
+          return CMD_ERROR;
+        }
+        if (ipmask_parse(cargv[*arg], &sin, &bits) == 0) {
+          reply(sender, "Error: Invalid CIDR mask supplied (for help, see help <searchcmd>)");
+          return CMD_ERROR;
+        }
+        *subset = (void *)refnode(iptree, &sin, bits);
+        (*arg)++;
+        break;
         
       default:
-       reply(sender,"Unrecognised flag -%c.",*ch);
+       reply(sender,"Unrecognised flag -%c. (for help, see help <searchcmd>)",*ch);
+        return CMD_ERROR;
       }
     }
   }
@@ -216,14 +409,15 @@ static int parseopts(int cargc, char **cargv, int *arg, int *limit, void **displ
   return CMD_OK;
 }
 
-void newsearch_ctxinit(searchCtx *ctx, searchParseFunc searchfn, replyFunc replyfn, wallFunc wallfn, void *arg, int type) {
+void newsearch_ctxinit(searchCtx *ctx, searchParseFunc searchfn, replyFunc replyfn, wallFunc wallfn, void *arg, searchCmd *cmd, nick *np) {
   memset(ctx, 0, sizeof(searchCtx));
   
   ctx->reply = replyfn;
   ctx->wall = wallfn;
   ctx->parser = searchfn;
   ctx->arg = arg;
-  ctx->type = type;
+  ctx->searchcmd = cmd;
+  ctx->sender = np;
 }
 
 int do_nicksearch_real(replyFunc reply, wallFunc wall, void *source, int cargc, char **cargv) {
@@ -235,10 +429,13 @@ int do_nicksearch_real(replyFunc reply, wallFunc wall, void *source, int cargc,
   searchCtx ctx;
   int ret;
 
-  if (cargc<1)
-    return CMD_USAGE;
-  
-  ret = parseopts(cargc, cargv, &arg, &limit, (void **)&display, nickOutputTree, reply, sender);
+  if (cargc<1) {
+    reply( sender, "Usage: [flags] <criteria>");
+    reply( sender, "For help, see help nicksearch");
+    return CMD_OK;
+  }
+  ret = parseopts(cargc, cargv, &arg, &limit, NULL, (void **)&display, reg_nicksearch->outputtree, reply, sender);
   if(ret != CMD_OK)
     return ret;
 
@@ -251,7 +448,7 @@ int do_nicksearch_real(replyFunc reply, wallFunc wall, void *source, int cargc,
     rejoinline(cargv[arg],cargc-arg);
   }
 
-  newsearch_ctxinit(&ctx, search_parse, reply, wall, NULL, SEARCHTYPE_NICK);
+  newsearch_ctxinit(&ctx, search_parse, reply, wall, NULL, reg_nicksearch, sender);
 
   if (!(search = ctx.parser(&ctx, cargv[arg]))) {
     reply(sender,"Parse error: %s",parseError);
@@ -322,10 +519,13 @@ int do_chansearch_real(replyFunc reply, wallFunc wall, void *source, int cargc,
   searchCtx ctx;
   int ret;
 
-  if (cargc<1)
-    return CMD_USAGE;
+  if (cargc<1) {
+    reply( sender, "Usage: [flags] <criteria>");
+    reply( sender, "For help, see help chansearch");
+    return CMD_OK;
+  }
   
-  ret = parseopts(cargc, cargv, &arg, &limit, (void **)&display, chanOutputTree, reply, sender);
+  ret = parseopts(cargc, cargv, &arg, &limit, NULL, (void **)&display, reg_chansearch->outputtree, reply, sender);
   if(ret != CMD_OK)
     return ret;
 
@@ -338,7 +538,7 @@ int do_chansearch_real(replyFunc reply, wallFunc wall, void *source, int cargc,
     rejoinline(cargv[arg],cargc-arg);
   }
 
-  newsearch_ctxinit(&ctx, search_parse, reply, wall, NULL, SEARCHTYPE_CHANNEL);
+  newsearch_ctxinit(&ctx, search_parse, reply, wall, NULL, reg_chansearch, sender);
   if (!(search = ctx.parser(&ctx, cargv[arg]))) {
     reply(sender,"Parse error: %s",parseError);
     return CMD_ERROR;
@@ -387,10 +587,13 @@ int do_usersearch_real(replyFunc reply, wallFunc wall, void *source, int cargc,
   searchCtx ctx;
   int ret;
 
-  if (cargc<1)
-    return CMD_USAGE;
-  
-  ret = parseopts(cargc, cargv, &arg, &limit, (void **)&display, userOutputTree, reply, sender);
+  if (cargc<1) {
+    reply( sender, "Usage: [flags] <criteria>");
+    reply( sender, "For help, see help usersearch");
+    return CMD_OK;
+  }
+  ret = parseopts(cargc, cargv, &arg, &limit, NULL, (void **)&display, reg_usersearch->outputtree, reply, sender);
   if(ret != CMD_OK)
     return ret;
 
@@ -403,7 +606,8 @@ int do_usersearch_real(replyFunc reply, wallFunc wall, void *source, int cargc,
     rejoinline(cargv[arg],cargc-arg);
   }
 
-  newsearch_ctxinit(&ctx, search_parse, reply, wall, NULL, SEARCHTYPE_USER);
+  newsearch_ctxinit(&ctx, search_parse, reply, wall, NULL, reg_usersearch, sender);
+
   if (!(search = ctx.parser(&ctx, cargv[arg]))) {
     reply(sender,"Parse error: %s",parseError);
     return CMD_ERROR;
@@ -751,10 +955,14 @@ struct searchNode *search_parse(searchCtx *ctx, char *input) {
     if (*(ch-1) == 0) /* if the last character was a space */
       j--; /* remove an argument */
     
-    if (!(cmd=findcommandintree(searchTree,argvector[0],1))) {
-      parseError = "Unknown command";
+    if (!(cmd=findcommandintree(ctx->searchcmd->searchtree,argvector[0],1))) {
+      parseError = "Unknown command (for valid command list, see help <searchcmd>)";
       return NULL;
     } else {
+      if ( !controlpermitted( cmd->level, ctx->sender) ) { 
+        parseError = "Access Denied (for valid command list, see help <searchcmd>)";
+        return NULL;
+      }
       return ((parseFunc)cmd->handler)(ctx, j, argvector+1);
     }
   } else {