/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:
plugins.d/plymouth.c
2
2
/*
3
3
 * Plymouth - Read a password from Plymouth and output it
4
4
 * 
5
 
 * Copyright © 2010-2011 Teddy Hogeborn
6
 
 * Copyright © 2010-2011 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
 */
75
76
}
76
77
 
77
78
/* Function to use when printing errors */
 
79
__attribute__((format (gnu_printf, 3, 4)))
78
80
void error_plus(int status, int errnum, const char *formatstring,
79
81
                ...){
80
82
  va_list ap;
83
85
  
84
86
  va_start(ap, formatstring);
85
87
  ret = vasprintf(&text, formatstring, ap);
86
 
  if (ret == -1){
 
88
  if(ret == -1){
87
89
    fprintf(stderr, "Mandos plugin %s: ",
88
90
            program_invocation_short_name);
89
91
    vfprintf(stderr, formatstring, ap);
153
155
  return true;
154
156
}
155
157
 
 
158
__attribute__((nonnull (2, 3)))
156
159
bool exec_and_wait(pid_t *pid_return, const char *path,
157
 
                   const char **argv, bool interruptable,
 
160
                   const char * const *argv, bool interruptable,
158
161
                   bool daemonize){
159
162
  int status;
160
163
  int ret;
172
175
      }
173
176
    }
174
177
    
175
 
    char **new_argv = NULL;
 
178
    char **new_argv = malloc(sizeof(const char *));
 
179
    if(new_argv == NULL){
 
180
      error_plus(0, errno, "malloc");
 
181
      _exit(EX_OSERR);
 
182
    }
176
183
    char **tmp;
177
184
    int i = 0;
178
 
    for (; argv[i]!=NULL; i++){
179
 
      tmp = realloc(new_argv, sizeof(const char *) * ((size_t)i + 1));
180
 
      if (tmp == NULL){
 
185
    for (; argv[i] != NULL; i++){
 
186
      tmp = realloc(new_argv, sizeof(const char *) * ((size_t)i + 2));
 
187
      if(tmp == NULL){
181
188
        error_plus(0, errno, "realloc");
182
189
        free(new_argv);
183
190
        _exit(EX_OSERR);
212
219
  return false;
213
220
}
214
221
 
 
222
__attribute__((nonnull))
215
223
int is_plymouth(const struct dirent *proc_entry){
216
224
  int ret;
217
225
  {
218
 
    uintmax_t maxvalue;
 
226
    uintmax_t proc_id;
219
227
    char *tmp;
220
228
    errno = 0;
221
 
    maxvalue = strtoumax(proc_entry->d_name, &tmp, 10);
 
229
    proc_id = strtoumax(proc_entry->d_name, &tmp, 10);
222
230
 
223
231
    if(errno != 0 or *tmp != '\0'
224
 
       or maxvalue != (uintmax_t)((pid_t)maxvalue)){
 
232
       or proc_id != (uintmax_t)((pid_t)proc_id)){
225
233
      return 0;
226
234
    }
227
235
  }
262
270
 
263
271
pid_t get_pid(void){
264
272
  int ret;
265
 
  uintmax_t maxvalue = 0;
 
273
  uintmax_t proc_id = 0;
266
274
  FILE *pidfile = fopen(plymouth_pid, "r");
267
275
  /* Try the new pid file location */
268
276
  if(pidfile != NULL){
269
 
    ret = fscanf(pidfile, "%" SCNuMAX, &maxvalue);
 
277
    ret = fscanf(pidfile, "%" SCNuMAX, &proc_id);
270
278
    if(ret != 1){
271
 
      maxvalue = 0;
 
279
      proc_id = 0;
272
280
    }
273
281
    fclose(pidfile);
274
282
  }
275
283
  /* Try the old pid file location */
276
 
  if(maxvalue == 0){
 
284
  if(proc_id == 0){
277
285
    pidfile = fopen(plymouth_pid, "r");
278
286
    if(pidfile != NULL){
279
 
      ret = fscanf(pidfile, "%" SCNuMAX, &maxvalue);
 
287
      ret = fscanf(pidfile, "%" SCNuMAX, &proc_id);
280
288
      if(ret != 1){
281
 
        maxvalue = 0;
 
289
        proc_id = 0;
282
290
      }
283
291
      fclose(pidfile);
284
292
    }
285
293
  }
286
294
  /* Look for a plymouth process */
287
 
  if(maxvalue == 0){
 
295
  if(proc_id == 0){
288
296
    struct dirent **direntries = NULL;
289
297
    ret = scandir("/proc", &direntries, is_plymouth, alphasort);
290
 
    if (ret == -1){
 
298
    if(ret == -1){
291
299
      error_plus(0, errno, "scandir");
292
300
    }
293
 
    if (ret > 0){
294
 
      ret = sscanf(direntries[0]->d_name, "%" SCNuMAX, &maxvalue);
295
 
      if (ret < 0){
 
301
    if(ret > 0){
 
302
      ret = sscanf(direntries[0]->d_name, "%" SCNuMAX, &proc_id);
 
303
      if(ret < 0){
296
304
        error_plus(0, errno, "sscanf");
297
305
      }
298
306
    }
301
309
    free(direntries);
302
310
  }
303
311
  pid_t pid;
304
 
  pid = (pid_t)maxvalue;
305
 
  if((uintmax_t)pid == maxvalue){
 
312
  pid = (pid_t)proc_id;
 
313
  if((uintmax_t)pid == proc_id){
306
314
    return pid;
307
315
  }
308
316
  
309
317
  return 0;
310
318
}
311
319
 
312
 
const char **getargv(pid_t pid){
 
320
char **getargv(pid_t pid){
313
321
  int cl_fd;
314
322
  char *cmdline_filename;
315
323
  ssize_t sret;
376
384
    return NULL;
377
385
  }
378
386
  argz_extract(cmdline, cmdline_len, argv); /* Create argv */
379
 
  return (const char **)argv;
 
387
  return argv;
380
388
}
381
389
 
382
390
int main(__attribute__((unused))int argc,
457
465
  }
458
466
  kill_and_wait(plymouth_command_pid);
459
467
  
460
 
  const char **plymouthd_argv;
 
468
  char **plymouthd_argv = NULL;
461
469
  pid_t pid = get_pid();
462
470
  if(pid == 0){
463
471
    error_plus(0, 0, "plymouthd pid not found");
464
 
    plymouthd_argv = plymouthd_default_argv;
465
472
  } else {
466
473
    plymouthd_argv = getargv(pid);
467
474
  }
470
477
                       { plymouth_path, "quit", NULL },
471
478
                       false, false);
472
479
  if(not bret){
 
480
    if(plymouthd_argv != NULL){
 
481
      free(*plymouthd_argv);
 
482
      free(plymouthd_argv);
 
483
    }
473
484
    exit(EXIT_FAILURE);
474
485
  }
475
 
  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,
476
490
                       false, true);
 
491
  if(plymouthd_argv != NULL){
 
492
    free(*plymouthd_argv);
 
493
    free(plymouthd_argv);
 
494
  }
477
495
  if(not bret){
478
496
    exit(EXIT_FAILURE);
479
497
  }