/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

  • Committer: Teddy Hogeborn
  • Date: 2018-04-01 19:31:31 UTC
  • Revision ID: teddy@recompile.se-20180401193131-w8but5r9ryne9et4
Print a boolean more readably in error message output

* plugins.d/mandos-client.c (pgp_packet_decrypt): If decryption fails,
  print wrong_key_usage as "Yes" or "No", not a number.

Show diffs side-by-side

added added

removed removed

Lines of Context:
9
9
 * "browse_callback", and parts of "main".
10
10
 * 
11
11
 * Everything else is
12
 
 * Copyright © 2008-2016 Teddy Hogeborn
13
 
 * Copyright © 2008-2016 Björn Påhlsson
14
 
 * 
15
 
 * This program is free software: you can redistribute it and/or
16
 
 * modify it under the terms of the GNU General Public License as
17
 
 * published by the Free Software Foundation, either version 3 of the
18
 
 * License, or (at your option) any later version.
19
 
 * 
20
 
 * This program is distributed in the hope that it will be useful, but
 
12
 * Copyright © 2008-2018 Teddy Hogeborn
 
13
 * Copyright © 2008-2018 Björn Påhlsson
 
14
 * 
 
15
 * This file is part of Mandos.
 
16
 * 
 
17
 * Mandos is free software: you can redistribute it and/or modify it
 
18
 * under the terms of the GNU General Public License as published by
 
19
 * the Free Software Foundation, either version 3 of the License, or
 
20
 * (at your option) any later version.
 
21
 * 
 
22
 * Mandos is distributed in the hope that it will be useful, but
21
23
 * WITHOUT ANY WARRANTY; without even the implied warranty of
22
24
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
23
25
 * General Public License for more details.
24
26
 * 
25
27
 * You should have received a copy of the GNU General Public License
26
 
 * along with this program.  If not, see
27
 
 * <http://www.gnu.org/licenses/>.
 
28
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
28
29
 * 
29
30
 * Contact the authors at <mandos@recompile.se>.
30
31
 */
419
420
      if(result == NULL){
420
421
        fprintf_plus(stderr, "gpgme_op_decrypt_result failed\n");
421
422
      } else {
422
 
        fprintf_plus(stderr, "Unsupported algorithm: %s\n",
423
 
                     result->unsupported_algorithm);
424
 
        fprintf_plus(stderr, "Wrong key usage: %u\n",
425
 
                     result->wrong_key_usage);
 
423
        if(result->unsupported_algorithm != NULL) {
 
424
          fprintf_plus(stderr, "Unsupported algorithm: %s\n",
 
425
                       result->unsupported_algorithm);
 
426
        }
 
427
        fprintf_plus(stderr, "Wrong key usage: %s\n",
 
428
                     result->wrong_key_usage ? "Yes" : "No");
426
429
        if(result->file_name != NULL){
427
430
          fprintf_plus(stderr, "File name: %s\n", result->file_name);
428
431
        }
612
615
        }
613
616
        params.size += (unsigned int)bytes_read;
614
617
      }
 
618
      ret = close(dhpfile);
 
619
      if(ret == -1){
 
620
        perror_plus("close");
 
621
      }
615
622
      if(params.data == NULL){
616
623
        dhparamsfilename = NULL;
617
624
      }
1494
1501
  return retval;
1495
1502
}
1496
1503
 
1497
 
__attribute__((nonnull))
1498
1504
static void resolve_callback(AvahiSServiceResolver *r,
1499
1505
                             AvahiIfIndex interface,
1500
1506
                             AvahiProtocol proto,
1655
1661
      perror_plus("ioctl SIOCGIFFLAGS");
1656
1662
      errno = old_errno;
1657
1663
    }
 
1664
    if((close(s) == -1) and debug){
 
1665
      old_errno = errno;
 
1666
      perror_plus("close");
 
1667
      errno = old_errno;
 
1668
    }
1658
1669
    return false;
1659
1670
  }
 
1671
  if((close(s) == -1) and debug){
 
1672
    old_errno = errno;
 
1673
    perror_plus("close");
 
1674
    errno = old_errno;
 
1675
  }
1660
1676
  return true;
1661
1677
}
1662
1678
 
1923
1939
      return;
1924
1940
    }
1925
1941
  }
 
1942
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
 
1943
  if(devnull == -1){
 
1944
    perror_plus("open(\"/dev/null\", O_RDONLY)");
 
1945
    return;
 
1946
  }
1926
1947
  int numhooks = scandirat(hookdir_fd, ".", &direntries,
1927
1948
                           runnable_hook, alphasort);
1928
1949
  if(numhooks == -1){
1929
1950
    perror_plus("scandir");
 
1951
    close(devnull);
1930
1952
    return;
1931
1953
  }
1932
1954
  struct dirent *direntry;
1933
1955
  int ret;
1934
 
  int devnull = (int)TEMP_FAILURE_RETRY(open("/dev/null", O_RDONLY));
1935
 
  if(devnull == -1){
1936
 
    perror_plus("open(\"/dev/null\", O_RDONLY)");
1937
 
    return;
1938
 
  }
1939
1956
  for(int i = 0; i < numhooks; i++){
1940
1957
    direntry = direntries[i];
1941
1958
    if(debug){
3061
3078
                                                | O_PATH));
3062
3079
    if(dir_fd == -1){
3063
3080
      perror_plus("open");
 
3081
      return;
3064
3082
    }
3065
3083
    int numentries = scandirat(dir_fd, ".", &direntries,
3066
3084
                               notdotentries, alphasort);
3083
3101
            clean_dir_at(dir_fd, direntries[i]->d_name, level+1);
3084
3102
            dret = 0;
3085
3103
          }
3086
 
          if(dret == -1){
 
3104
          if((dret == -1) and (errno != ENOENT)){
3087
3105
            fprintf_plus(stderr, "unlink(\"%s/%s\"): %s\n", dirname,
3088
3106
                         direntries[i]->d_name, strerror(errno));
3089
3107
          }
3093
3111
      
3094
3112
      /* need to clean even if 0 because man page doesn't specify */
3095
3113
      free(direntries);
3096
 
      if(numentries == -1){
3097
 
        perror_plus("scandirat");
3098
 
      }
3099
3114
      dret = unlinkat(base, dirname, AT_REMOVEDIR);
3100
3115
      if(dret == -1 and errno != ENOENT){
3101
3116
        perror_plus("rmdir");