/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: 2016-06-03 17:27:03 UTC
  • Revision ID: teddy@recompile.se-20160603172703-mc6tjor6rhq4xy74
mandos: Bug fix: Do multiprocessing cleanup correctly on exit

* mandos (main): Save module "multiprocessing" and open file "wnull"
                 as scope variables accessible by function cleanup(),
                 since the module and global variable may not be
                 accessible when the cleanup() function is run as
                 scheduled by atexit().

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
2
2
/*
3
3
 * Mandos plugin runner - Run Mandos plugins
4
4
 *
5
 
 * Copyright © 2008-2018 Teddy Hogeborn
6
 
 * Copyright © 2008-2018 Björn Påhlsson
7
 
 * 
8
 
 * This file is part of Mandos.
9
 
 * 
10
 
 * Mandos is free software: you can redistribute it and/or modify it
11
 
 * under the terms of the GNU General Public License as published by
12
 
 * the Free Software Foundation, either version 3 of the License, or
13
 
 * (at your option) any later version.
14
 
 * 
15
 
 * Mandos is distributed in the hope that it will be useful, but
 
5
 * Copyright © 2008-2016 Teddy Hogeborn
 
6
 * Copyright © 2008-2016 Björn Påhlsson
 
7
 * 
 
8
 * This program is free software: you can redistribute it and/or
 
9
 * modify it under the terms of the GNU General Public License as
 
10
 * published by the Free Software Foundation, either version 3 of the
 
11
 * License, or (at your option) any later version.
 
12
 * 
 
13
 * This program is distributed in the hope that it will be useful, but
16
14
 * WITHOUT ANY WARRANTY; without even the implied warranty of
17
15
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
18
16
 * General Public License for more details.
19
17
 * 
20
18
 * You should have received a copy of the GNU General Public License
21
 
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
 
19
 * along with this program.  If not, see
 
20
 * <http://www.gnu.org/licenses/>.
22
21
 * 
23
22
 * Contact the authors at <mandos@recompile.se>.
24
23
 */
313
312
__attribute__((nonnull))
314
313
static void free_plugin(plugin *plugin_node){
315
314
  
316
 
  for(char **arg = (plugin_node->argv)+1; *arg != NULL; arg++){
 
315
  for(char **arg = plugin_node->argv; *arg != NULL; arg++){
317
316
    free(*arg);
318
317
  }
319
 
  free(plugin_node->name);
320
318
  free(plugin_node->argv);
321
319
  for(char **env = plugin_node->environ; *env != NULL; env++){
322
320
    free(*env);
565
563
    case '?':                   /* --help */
566
564
      state->flags &= ~(unsigned int)ARGP_NO_EXIT; /* force exit */
567
565
      argp_state_help(state, state->out_stream, ARGP_HELP_STD_HELP);
568
 
      __builtin_unreachable();
569
566
    case -3:                    /* --usage */
570
567
      state->flags &= ~(unsigned int)ARGP_NO_EXIT; /* force exit */
571
568
      argp_state_help(state, state->out_stream,
572
569
                      ARGP_HELP_USAGE | ARGP_HELP_EXIT_OK);
573
 
      __builtin_unreachable();
574
570
    case 'V':                   /* --version */
575
571
      fprintf(state->out_stream, "%s\n", argp_program_version);
576
572
      exit(EXIT_SUCCESS);
586
582
      if(arg[0] == '\0'){
587
583
        break;
588
584
      }
589
 
#if __GNUC__ >= 7
590
 
      __attribute__((fallthrough));
591
 
#else
592
 
          /* FALLTHROUGH */
593
 
#endif
594
585
    default:
595
586
      return ARGP_ERR_UNKNOWN;
596
587
    }
801
792
  }
802
793
  
803
794
  if(debug){
804
 
    for(plugin *p = plugin_list; p != NULL; p = p->next){
 
795
    for(plugin *p = plugin_list; p != NULL; p=p->next){
805
796
      fprintf(stderr, "Plugin: %s has %d arguments\n",
806
797
              p->name ? p->name : "Global", p->argc - 1);
807
798
      for(char **a = p->argv; *a != NULL; a++){
816
807
  
817
808
  if(getuid() == 0){
818
809
    /* Work around Debian bug #633582:
819
 
       <https://bugs.debian.org/633582> */
 
810
       <http://bugs.debian.org/633582> */
820
811
    int plugindir_fd = open(/* plugindir or */ PDIR, O_RDONLY);
821
812
    if(plugindir_fd == -1){
822
813
      if(errno != ENOENT){
1100
1091
    
1101
1092
    new_plugin->pid = pid;
1102
1093
    new_plugin->fd = pipefd[0];
1103
 
 
1104
 
    if(debug){
1105
 
      fprintf(stderr, "Plugin %s started (PID %" PRIdMAX ")\n",
1106
 
              new_plugin->name, (intmax_t) (new_plugin->pid));
1107
 
    }
1108
 
 
 
1094
    
1109
1095
    /* Unblock SIGCHLD so signal handler can be run if this process
1110
1096
       has already completed */
1111
1097
    ret = (int)TEMP_FAILURE_RETRY(sigprocmask(SIG_UNBLOCK,