/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/mandos-client.c

* network-hooks.d/bridge: Use "/usr/sbin/brctl" explicitly.
* plugins.d/mandos-client.c (run_network_hooks): Raise priviliges in
                                                 child process.
  (main): Do not use getuid() to check if running setuid root.  Do not
          raise privileges for run_network_hooks.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
170
170
 
171
171
/* Function to use when printing errors */
172
172
void perror_plus(const char *print_text){
173
 
  int e = errno;
174
173
  fprintf(stderr, "Mandos plugin %s: ",
175
174
          program_invocation_short_name);
176
 
  errno = e;
177
175
  perror(print_text);
178
176
}
179
177
 
180
 
__attribute__((format (gnu_printf, 2, 3)))
181
178
int fprintf_plus(FILE *stream, const char *format, ...){
182
179
  va_list ap;
183
180
  va_start (ap, format);
205
202
}
206
203
 
207
204
/* Add server to set of servers to retry periodically */
208
 
bool add_server(const char *ip, uint16_t port, AvahiIfIndex if_index,
209
 
                int af){
 
205
int add_server(const char *ip, uint16_t port, AvahiIfIndex if_index,
 
206
               int af){
210
207
  int ret;
211
208
  server *new_server = malloc(sizeof(server));
212
209
  if(new_server == NULL){
213
210
    perror_plus("malloc");
214
 
    return false;
 
211
    return -1;
215
212
  }
216
213
  *new_server = (server){ .ip = strdup(ip),
217
214
                          .port = port,
219
216
                          .af = af };
220
217
  if(new_server->ip == NULL){
221
218
    perror_plus("strdup");
222
 
    return false;
 
219
    return -1;
223
220
  }
224
221
  /* Special case of first server */
225
222
  if (mc.current_server == NULL){
236
233
  ret = clock_gettime(CLOCK_MONOTONIC, &mc.current_server->last_seen);
237
234
  if(ret == -1){
238
235
    perror_plus("clock_gettime");
239
 
    return false;
 
236
    return -1;
240
237
  }
241
 
  return true;
 
238
  return 0;
242
239
}
243
240
 
244
241
/* 
1033
1030
      if(ret == 0){
1034
1031
        avahi_simple_poll_quit(mc.simple_poll);
1035
1032
      } else {
1036
 
        if(not add_server(ip, port, interface,
1037
 
                          avahi_proto_to_af(proto))){
1038
 
          fprintf_plus(stderr, "Failed to add server \"%s\" to server"
1039
 
                       " list\n", name);
1040
 
        }
 
1033
        ret = add_server(ip, port, interface,
 
1034
                         avahi_proto_to_af(proto));
1041
1035
      }
1042
1036
    }
1043
1037
  }
1437
1431
        if(ret == -1){
1438
1432
          perror_plus("setgroups");
1439
1433
        }
 
1434
        fprintf_plus(stderr, "Child: getuid() = %d\n", getuid());
 
1435
        fprintf_plus(stderr, "Child: geteuid() = %d\n", geteuid());
1440
1436
        dup2(devnull, STDIN_FILENO);
1441
1437
        close(devnull);
1442
1438
        dup2(STDERR_FILENO, STDOUT_FILENO);
1473
1469
          _exit(EX_OSERR);
1474
1470
        }
1475
1471
        free(delaystring);
1476
 
        if(execl(fullname, direntry->d_name, mode, NULL) == -1){
1477
 
          perror_plus("execl");
1478
 
          _exit(EXIT_FAILURE);
1479
 
        }
 
1472
        ret = execl(fullname, direntry->d_name, mode, NULL);
 
1473
        perror_plus("execl");
1480
1474
      } else {
1481
1475
        int status;
1482
1476
        if(TEMP_FAILURE_RETRY(waitpid(hook_pid, &status, 0)) == -1){
1679
1673
        argp_state_help(state, state->out_stream,
1680
1674
                        ARGP_HELP_USAGE | ARGP_HELP_EXIT_ERR);
1681
1675
      case 'V':                 /* --version */
 
1676
        fprintf_plus(state->out_stream,
 
1677
                     "Mandos plugin mandos-client: ");
1682
1678
        fprintf_plus(state->out_stream, "%s\n", argp_program_version);
1683
1679
        exit(argp_err_exit_status);
1684
1680
        break;