Remove two useless NULL pointer checks in load_one_re_filterfile().
authorFabian Keil <fk@fabiankeil.de>
Sat, 30 Jul 2011 15:12:42 +0000 (15:12 +0000)
committerFabian Keil <fk@fabiankeil.de>
Sat, 30 Jul 2011 15:12:42 +0000 (15:12 +0000)
csp has already been dereferenced earlier and there's
no reason why it should be NULL anyway.

loaders.c

index c62a399..92d793c 100644 (file)
--- a/loaders.c
+++ b/loaders.c
@@ -1,4 +1,4 @@
-const char loaders_rcs[] = "$Id: loaders.c,v 1.81 2011/03/03 14:38:36 fabiankeil Exp $";
+const char loaders_rcs[] = "$Id: loaders.c,v 1.82 2011/04/19 13:00:47 fabiankeil Exp $";
 /*********************************************************************
  *
  * File        :  $Source: /cvsroot/ijbswa/current/loaders.c,v $
@@ -1132,10 +1132,7 @@ int load_one_re_filterfile(struct client_state *csp, int fileid)
     */
    if (!check_file_changed(current_re_filterfile[fileid], csp->config->re_filterfile[fileid], &fs))
    {
-      if (csp)
-      {
-         csp->rlist[fileid] = current_re_filterfile[fileid];
-      }
+      csp->rlist[fileid] = current_re_filterfile[fileid];
       return(0);
    }
    if (!fs)
@@ -1333,11 +1330,7 @@ int load_one_re_filterfile(struct client_state *csp, int fileid)
    fs->next    = files->next;
    files->next = fs;
    current_re_filterfile[fileid] = fs;
-
-   if (csp)
-   {
-      csp->rlist[fileid] = fs;
-   }
+   csp->rlist[fileid] = fs;
 
    return( 0 );