/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: 2014-03-23 19:24:40 UTC
  • Revision ID: teddy@recompile.se-20140323192440-d71iiqxebsxf9u2v
Update GCC warning flags and function attributes to GCC 4.7.

* Makefile (WARN): Update to include almost all warning flags.
* plugin-runner.c (getplugin, add_to_char_array, add_argument,
                   add_environment, set_cloexec_flag,
                   print_out_password): Add attribute
                                        "warn_unused_result".
  (main/parse_opt): Bug fix: Add error checking to --global-env,
                    --env-for, --plugin-dir, and --config-file, and
                    make sure errno does not "leak" from unrelated
                    functions.
* plugins.d/mandos-client.c
  (fprintf_plus, debuggnutls, resolve_callback): Add "nonnull"
                                                 attribute.
  (incbuffer, add_server, init_gpgme): Add "nonnull" and
                                       "warn_unused_result"
                                       attributes.
  (pgp_packet_decrypt, init_gnutls_global): - '' -
  (init_gnutls_session start_mandos_communication, get_flags): - '' -
  (good_flags, good_interface, interface_is_up): - '' -
  (interface_is_running, runnable_hook): - '' -
  (avahi_loop_with_timeout, bring_up_interface): : - '' -
  (safer_gnutls_strerror): Add "warn_unused_result" attribute.
  (notdotentries): Set "nonnull", "pure", and "warn_unused_result"
                   attributes.
  (raise_privileges, raise_privileges_permanently, lower_privileges,
  lower_privileges_permanently): Set "warn_unused_result" attribute.
  (run_network_hooks): Exit child process if it fails to do anything
                       it needs to do.  Make explicit cast to double
                       when passing float value to asprintf().  Change
                       return type to void - all callers changed.
  (bring_up_interface): Move variables "sd", "ret_errno", and
                        "ret_setflags" to innermost scope.  Bug fix:
                        Fail if could not get interface flags also in
                        non-debug mode, and restore old errno
                        correctly.  Print message if could not raise
                        (or later lower) privileges.
  (take_down_interface): Bug fix: When failing because it could not
                         get interface flags, restore old errno
                         correctly.  Print message if it could not
                         raise (or later lower) privileges.
  (main): Complain if failed to raise or lower privileges.  Only run
          network hooks or lower privileges if raising privileges was
          successful.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
105
105
 
106
106
/* Gets an existing plugin based on name,
107
107
   or if none is found, creates a new one */
 
108
__attribute__((warn_unused_result))
108
109
static plugin *getplugin(char *name){
109
110
  /* Check for existing plugin with that name */
110
111
  for(plugin *p = plugin_list; p != NULL; p = p->next){
171
172
}
172
173
 
173
174
/* Helper function for add_argument and add_environment */
174
 
__attribute__((nonnull))
 
175
__attribute__((nonnull, warn_unused_result))
175
176
static bool add_to_char_array(const char *new, char ***array,
176
177
                              int *len){
177
178
  /* Resize the pointed-to array to hold one more pointer */
202
203
}
203
204
 
204
205
/* Add to a plugin's argument vector */
205
 
__attribute__((nonnull(2)))
 
206
__attribute__((nonnull(2), warn_unused_result))
206
207
static bool add_argument(plugin *p, const char *arg){
207
208
  if(p == NULL){
208
209
    return false;
211
212
}
212
213
 
213
214
/* Add to a plugin's environment */
214
 
__attribute__((nonnull(2)))
 
215
__attribute__((nonnull(2), warn_unused_result))
215
216
static bool add_environment(plugin *p, const char *def, bool replace){
216
217
  if(p == NULL){
217
218
    return false;
244
245
 * Descriptor Flags".
245
246
 | [[info:libc:Descriptor%20Flags][File Descriptor Flags]] |
246
247
 */
 
248
__attribute__((warn_unused_result))
247
249
static int set_cloexec_flag(int fd){
248
250
  int ret = (int)TEMP_FAILURE_RETRY(fcntl(fd, F_GETFD, 0));
249
251
  /* If reading the flags failed, return error indication now. */
291
293
}
292
294
 
293
295
/* Prints out a password to stdout */
294
 
__attribute__((nonnull))
 
296
__attribute__((nonnull, warn_unused_result))
295
297
static bool print_out_password(const char *buffer, size_t length){
296
298
  ssize_t ret;
297
299
  for(size_t written = 0; written < length; written += (size_t)ret){
437
439
            break;
438
440
          }
439
441
        }
 
442
        errno = 0;
440
443
      }
441
444
      break;
442
445
    case 'G':                   /* --global-env */
443
 
      add_environment(getplugin(NULL), arg, true);
 
446
      if(add_environment(getplugin(NULL), arg, true)){
 
447
        errno = 0;
 
448
      }
444
449
      break;
445
450
    case 'o':                   /* --options-for */
446
451
      {
463
468
            break;
464
469
          }
465
470
        }
 
471
        errno = 0;
466
472
      }
467
473
      break;
468
474
    case 'E':                   /* --env-for */
480
486
          errno = EINVAL;
481
487
          break;
482
488
        }
483
 
        add_environment(getplugin(arg), envdef, true);
 
489
        if(add_environment(getplugin(arg), envdef, true)){
 
490
          errno = 0;
 
491
        }
484
492
      }
485
493
      break;
486
494
    case 'd':                   /* --disable */
488
496
        plugin *p = getplugin(arg);
489
497
        if(p != NULL){
490
498
          p->disabled = true;
 
499
          errno = 0;
491
500
        }
492
501
      }
493
502
      break;
496
505
        plugin *p = getplugin(arg);
497
506
        if(p != NULL){
498
507
          p->disabled = false;
 
508
          errno = 0;
499
509
        }
500
510
      }
501
511
      break;
502
512
    case 128:                   /* --plugin-dir */
503
513
      free(plugindir);
504
514
      plugindir = strdup(arg);
 
515
      if(plugindir != NULL){
 
516
        errno = 0;
 
517
      }
505
518
      break;
506
519
    case 129:                   /* --config-file */
507
520
      /* This is already done by parse_opt_config_file() */
515
528
        break;
516
529
      }
517
530
      uid = (uid_t)tmp_id;
 
531
      errno = 0;
518
532
      break;
519
533
    case 131:                   /* --groupid */
520
534
      tmp_id = strtoimax(arg, &tmp, 10);
525
539
        break;
526
540
      }
527
541
      gid = (gid_t)tmp_id;
 
542
      errno = 0;
528
543
      break;
529
544
    case 132:                   /* --debug */
530
545
      debug = true;
578
593
    case 129:                   /* --config-file */
579
594
      free(argfile);
580
595
      argfile = strdup(arg);
 
596
      if(argfile != NULL){
 
597
        errno = 0;
 
598
      }
581
599
      break;
582
600
    case 130:                   /* --userid */
583
601
    case 131:                   /* --groupid */