/mandos/trunk

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/trunk

« back to all changes in this revision

Viewing changes to plugin-runner.c

  • Committer: Teddy Hogeborn
  • Date: 2009-02-05 03:15:43 UTC
  • Revision ID: teddy@fukt.bsnet.se-20090205031543-cyd3j09i6iut38w1
* plugin-runner.c (struct plugin.status): Changed type to "int".
  (main): Check "proc->completed" before "proc->eof", since
          "completed" is of type "sig_atomic_t".

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
223
223
/* Mark processes as completed when they exit, and save their exit
224
224
   status. */
225
225
static void handle_sigchld(__attribute__((unused)) int sig){
226
 
  int old_errno = errno;
227
226
  while(true){
228
227
    plugin *proc = plugin_list;
229
228
    int status;
251
250
    proc->status = status;
252
251
    proc->completed = 1;
253
252
  }
254
 
  errno = old_errno;
255
253
}
256
254
 
257
255
/* Prints out a password to stdout */
664
662
  }
665
663
  
666
664
  /* Strip permissions down to nobody */
 
665
  ret = setuid(uid);
 
666
  if(ret == -1){
 
667
    perror("setuid");
 
668
  }  
667
669
  setgid(gid);
668
670
  if(ret == -1){
669
671
    perror("setgid");
670
672
  }
671
 
  ret = setuid(uid);
672
 
  if(ret == -1){
673
 
    perror("setuid");
674
 
  }
675
673
  
676
674
  if(plugindir == NULL){
677
675
    dir = opendir(PDIR);
936
934
  
937
935
  closedir(dir);
938
936
  dir = NULL;
939
 
  free_plugin(getplugin(NULL));
940
937
  
941
938
  for(plugin *p = plugin_list; p != NULL; p = p->next){
942
939
    if(p->pid != 0){
970
967
 
971
968
          if(debug){
972
969
            if(WIFEXITED(proc->status)){
973
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] exited with"
974
 
                      " status %d\n", proc->name,
975
 
                      (intmax_t) (proc->pid),
 
970
              fprintf(stderr, "Plugin %" PRIdMAX " exited with status"
 
971
                      " %d\n", (intmax_t) (proc->pid),
976
972
                      WEXITSTATUS(proc->status));
977
973
            } else if(WIFSIGNALED(proc->status)){
978
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] killed by"
979
 
                      " signal %d\n", proc->name,
980
 
                      (intmax_t) (proc->pid),
 
974
              fprintf(stderr, "Plugin %" PRIdMAX " killed by signal"
 
975
                      " %d\n", (intmax_t) (proc->pid),
981
976
                      WTERMSIG(proc->status));
982
977
            } else if(WCOREDUMP(proc->status)){
983
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] dumped"
984
 
                      " core\n", proc->name, (intmax_t) (proc->pid));
 
978
              fprintf(stderr, "Plugin %" PRIdMAX " dumped core\n",
 
979
                      (intmax_t) (proc->pid));
985
980
            }
986
981
          }
987
982