/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 plugins.d/plymouth.c

  • Committer: Teddy Hogeborn
  • Date: 2017-09-03 08:26:36 UTC
  • Revision ID: teddy@recompile.se-20170903082636-fo3qw1uoahjxw0by
Fix memory leak in plymouth plugin

* plugins.d/plymouth.c (getargv): Change return type to non-const.
  (main/plymouth_args): Change type to non-const and default to NULL.
  (main): Don't change plymouthd_argv, instead conditionally use it if
  it is allocated or use plymouthd_default_argv if not.
  (main): Free plymouthd_argv, if allocated.

Show diffs side-by-side

added added

removed removed

Lines of Context:
2
2
/*
3
3
 * Plymouth - Read a password from Plymouth and output it
4
4
 * 
5
 
 * Copyright © 2010-2016 Teddy Hogeborn
6
 
 * Copyright © 2010-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
 
5
 * Copyright © 2010-2017 Teddy Hogeborn
 
6
 * Copyright © 2010-2017 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
14
16
 * WITHOUT ANY WARRANTY; without even the implied warranty of
15
17
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
16
18
 * General Public License for more details.
17
19
 * 
18
20
 * You should have received a copy of the GNU General Public License
19
 
 * along with this program.  If not, see
20
 
 * <http://www.gnu.org/licenses/>.
 
21
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
21
22
 * 
22
23
 * Contact the authors at <mandos@recompile.se>.
23
24
 */
181
182
    }
182
183
    char **tmp;
183
184
    int i = 0;
184
 
    for (; argv[i]!=NULL; i++){
 
185
    for (; argv[i] != NULL; i++){
185
186
      tmp = realloc(new_argv, sizeof(const char *) * ((size_t)i + 2));
186
187
      if(tmp == NULL){
187
188
        error_plus(0, errno, "realloc");
316
317
  return 0;
317
318
}
318
319
 
319
 
const char * const * getargv(pid_t pid){
 
320
char **getargv(pid_t pid){
320
321
  int cl_fd;
321
322
  char *cmdline_filename;
322
323
  ssize_t sret;
383
384
    return NULL;
384
385
  }
385
386
  argz_extract(cmdline, cmdline_len, argv); /* Create argv */
386
 
  return (const char * const *)argv;
 
387
  return argv;
387
388
}
388
389
 
389
390
int main(__attribute__((unused))int argc,
464
465
  }
465
466
  kill_and_wait(plymouth_command_pid);
466
467
  
467
 
  const char * const *plymouthd_argv;
 
468
  char **plymouthd_argv = NULL;
468
469
  pid_t pid = get_pid();
469
470
  if(pid == 0){
470
471
    error_plus(0, 0, "plymouthd pid not found");
471
 
    plymouthd_argv = plymouthd_default_argv;
472
472
  } else {
473
473
    plymouthd_argv = getargv(pid);
474
474
  }
477
477
                       { plymouth_path, "quit", NULL },
478
478
                       false, false);
479
479
  if(not bret){
 
480
    if(plymouthd_argv != NULL){
 
481
      free(*plymouthd_argv);
 
482
      free(plymouthd_argv);
 
483
    }
480
484
    exit(EXIT_FAILURE);
481
485
  }
482
 
  bret = exec_and_wait(NULL, plymouthd_path, plymouthd_argv,
 
486
  bret = exec_and_wait(NULL, plymouthd_path,
 
487
                       (plymouthd_argv != NULL)
 
488
                       ? (const char * const *)plymouthd_argv
 
489
                       : plymouthd_default_argv,
483
490
                       false, true);
 
491
  if(plymouthd_argv != NULL){
 
492
    free(*plymouthd_argv);
 
493
    free(plymouthd_argv);
 
494
  }
484
495
  if(not bret){
485
496
    exit(EXIT_FAILURE);
486
497
  }