/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to plugins.d/mandos-client.c

* README (The Plugin System): Removed redundant text about options and
                              files for the plugins, this is now
                              documented in the manuals for the
                              plugins.

* plugins.d/mandos-client.c (main): Remove comment which was copied
                                    from another program by mistake.
                                    Use "sscanf" instead of "strtol"
                                    to parse numbers; this uses the
                                    correct type instead of casting.
                                    Don't report errors when removing
                                    temporary directory if directory
                                    is already gone.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
1
1
/*  -*- coding: utf-8 -*- */
2
2
/*
3
 
 * Mandos client - get and decrypt data from a Mandos server
 
3
 * Mandos-client - get and decrypt data from a Mandos server
4
4
 *
5
5
 * This program is partly derived from an example program for an Avahi
6
6
 * service browser, downloaded from
9
9
 * "browse_callback", and parts of "main".
10
10
 * 
11
11
 * Everything else is
12
 
 * Copyright © 2008 Teddy Hogeborn & Björn Påhlsson
 
12
 * Copyright © 2008,2009 Teddy Hogeborn
 
13
 * Copyright © 2008,2009 Björn Påhlsson
13
14
 * 
14
15
 * This program is free software: you can redistribute it and/or
15
16
 * modify it under the terms of the GNU General Public License as
35
36
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY(), asprintf() */
36
37
 
37
38
#include <stdio.h>              /* fprintf(), stderr, fwrite(),
38
 
                                   stdout, ferror() */
 
39
                                   stdout, ferror(), sscanf */
39
40
#include <stdint.h>             /* uint16_t, uint32_t */
40
41
#include <stddef.h>             /* NULL, size_t, ssize_t */
41
42
#include <stdlib.h>             /* free(), EXIT_SUCCESS, EXIT_FAILURE,
54
55
                                   connect() */
55
56
#include <fcntl.h>              /* open() */
56
57
#include <dirent.h>             /* opendir(), struct dirent, readdir() */
57
 
#include <inttypes.h>           /* PRIu16 */
 
58
#include <inttypes.h>           /* PRIu16, SCNu16 */
58
59
#include <assert.h>             /* assert() */
59
60
#include <errno.h>              /* perror(), errno */
60
61
#include <time.h>               /* time() */
66
67
                                   getuid(), getgid(), setuid(),
67
68
                                   setgid() */
68
69
#include <arpa/inet.h>          /* inet_pton(), htons */
69
 
#include <iso646.h>             /* not, and */
 
70
#include <iso646.h>             /* not, and, or */
70
71
#include <argp.h>               /* struct argp_option, error_t, struct
71
72
                                   argp_state, struct argp,
72
73
                                   argp_parse(), ARGP_KEY_ARG,
155
156
    int fd;
156
157
    gpgme_data_t pgp_data;
157
158
    
158
 
    fd = TEMP_FAILURE_RETRY(open(filename, O_RDONLY));
 
159
    fd = (int)TEMP_FAILURE_RETRY(open(filename, O_RDONLY));
159
160
    if(fd == -1){
160
161
      perror("open");
161
162
      return false;
175
176
      return false;
176
177
    }
177
178
    
178
 
    ret = TEMP_FAILURE_RETRY(close(fd));
 
179
    ret = (int)TEMP_FAILURE_RETRY(close(fd));
179
180
    if(ret == -1){
180
181
      perror("close");
181
182
    }
500
501
                                      AvahiIfIndex if_index,
501
502
                                      mandos_context *mc){
502
503
  int ret, tcp_sd;
 
504
  ssize_t sret;
503
505
  union { struct sockaddr in; struct sockaddr_in6 in6; } to;
504
506
  char *buffer = NULL;
505
507
  char *decrypted_buffer;
576
578
  written = 0;
577
579
  while (true){
578
580
    size_t out_size = strlen(out);
579
 
    ret = TEMP_FAILURE_RETRY(write(tcp_sd, out + written,
 
581
    ret = (int)TEMP_FAILURE_RETRY(write(tcp_sd, out + written,
580
582
                                   out_size - written));
581
583
    if (ret == -1){
582
584
      perror("write");
631
633
      goto mandos_end;
632
634
    }
633
635
    
634
 
    ret = gnutls_record_recv(session, buffer+buffer_length,
635
 
                             BUFFER_SIZE);
636
 
    if (ret == 0){
 
636
    sret = gnutls_record_recv(session, buffer+buffer_length,
 
637
                              BUFFER_SIZE);
 
638
    if (sret == 0){
637
639
      break;
638
640
    }
639
 
    if (ret < 0){
640
 
      switch(ret){
 
641
    if (sret < 0){
 
642
      switch(sret){
641
643
      case GNUTLS_E_INTERRUPTED:
642
644
      case GNUTLS_E_AGAIN:
643
645
        break;
660
662
        goto mandos_end;
661
663
      }
662
664
    } else {
663
 
      buffer_length += (size_t) ret;
 
665
      buffer_length += (size_t) sret;
664
666
    }
665
667
  }
666
668
  
702
704
  
703
705
 mandos_end:
704
706
  free(buffer);
705
 
  ret = TEMP_FAILURE_RETRY(close(tcp_sd));
 
707
  ret = (int)TEMP_FAILURE_RETRY(close(tcp_sd));
706
708
  if(ret == -1){
707
709
    perror("close");
708
710
  }
864
866
      
865
867
      error_t parse_opt (int key, char *arg,
866
868
                         struct argp_state *state) {
867
 
        /* Get the INPUT argument from `argp_parse', which we know is
868
 
           a pointer to our plugin list pointer. */
869
869
        switch (key) {
870
870
        case 128:               /* --debug */
871
871
          debug = true;
883
883
          pubkey = arg;
884
884
          break;
885
885
        case 129:               /* --dh-bits */
886
 
          errno = 0;
887
 
          mc.dh_bits = (unsigned int) strtol(arg, NULL, 10);
888
 
          if (errno){
889
 
            perror("strtol");
 
886
          ret = sscanf(arg, "%u", &mc.dh_bits);
 
887
          if(ret == 0 or mc.dh_bits == 0){
 
888
            fprintf(stderr, "Bad number of DH bits\n");
890
889
            exit(EXIT_FAILURE);
891
890
          }
892
891
          break;
939
938
          goto end;
940
939
        }
941
940
      }
942
 
      ret = TEMP_FAILURE_RETRY(close(sd));
 
941
      ret = (int)TEMP_FAILURE_RETRY(close(sd));
943
942
      if(ret == -1){
944
943
        perror("close");
945
944
      }
996
995
        exitcode = EXIT_FAILURE;
997
996
        goto end;
998
997
      }
999
 
      errno = 0;
1000
 
      uint16_t port = (uint16_t) strtol(address+1, NULL, 10);
1001
 
      if(errno){
1002
 
        perror("Bad port number");
 
998
      uint16_t port;
 
999
      ret = sscanf(address+1, "%" SCNu16, &port);
 
1000
      if(ret == 0 or port == 0){
 
1001
        fprintf(stderr, "Bad port number\n");
1003
1002
        exitcode = EXIT_FAILURE;
1004
1003
        goto end;
1005
1004
      }
1108
1107
      struct dirent *direntry;
1109
1108
      d = opendir(tempdir);
1110
1109
      if(d == NULL){
1111
 
        perror("opendir");
 
1110
        if(errno != ENOENT){
 
1111
          perror("opendir");
 
1112
        }
1112
1113
      } else {
1113
1114
        while(true){
1114
1115
          direntry = readdir(d);
1134
1135
        closedir(d);
1135
1136
      }
1136
1137
      ret = rmdir(tempdir);
1137
 
      if(ret == -1){
 
1138
      if(ret == -1 and errno != ENOENT){
1138
1139
        perror("rmdir");
1139
1140
      }
1140
1141
    }