Set FD_CLOEXEC flag on all sockets.
[tinc] / src / conf.c
index eb13c42..1560541 100644 (file)
@@ -1,9 +1,10 @@
 /*
     conf.c -- configuration code
     Copyright (C) 1998 Robert van der Meulen
-                  1998-2001 Ivo Timmermans <itimmermans@bigfoot.com>
-                  2000,2001 Guus Sliepen <guus@sliepen.warande.net>
-                 2000 Cris van Pelt <tribbel@arise.dhs.org>
+                  1998-2005 Ivo Timmermans
+                  2000-2010 Guus Sliepen <guus@tinc-vpn.org>
+                  2010-2011 Julien Muchembled <jm@jmuchemb.eu>
+                 2000 Cris van Pelt
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
     GNU General Public License for more details.
 
-    You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
-
-    $Id: conf.c,v 1.9.4.49 2001/11/16 12:17:03 zarq Exp $
+    You should have received a copy of the GNU General Public License along
+    with this program; if not, write to the Free Software Foundation, Inc.,
+    51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */
 
-#include "config.h"
-
-#include <ctype.h>
-#include <errno.h>
-#include <netdb.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-#include <syslog.h>
-#include <sys/stat.h>
-#include <sys/types.h>
-#include <unistd.h>
-#include <syslog.h>
-#include <string.h>
-
-#include <xalloc.h>
-#include <utils.h> /* for cp */
-#include <avl_tree.h>
+#include "system.h"
 
+#include "avl_tree.h"
+#include "connection.h"
 #include "conf.h"
-
-#include "system.h"
+#include "list.h"
+#include "logger.h"
+#include "netutl.h"                            /* for str2address */
+#include "protocol.h"
+#include "utils.h"                             /* for cp */
+#include "xalloc.h"
 
 avl_tree_t *config_tree;
 
-int debug_lvl = 0;
-int timeout = 0; /* seconds before timeout */
-char *confbase = NULL;           /* directory in which all config files are */
-char *netname = NULL;            /* name of the vpn network */
-
-/* Will be set if HUP signal is received. It will be processed when it is safe. */
-int sighup = 0;
-
-int config_compare(config_t *a, config_t *b)
-{
-  int result;
-  
-  result = strcmp(a->variable, b->variable);
-  
-  if(result)
-    return result;
-
-  result = a->line - b->line;
-  
-  if(result)
-    return result;
-  else
-    return strcmp(a->file, b->file);
+int pinginterval = 0;                  /* seconds between pings */
+int pingtimeout = 0;                   /* seconds to wait for response */
+char *confbase = NULL;                 /* directory in which all config files are */
+char *netname = NULL;                  /* name of the vpn network */
+list_t *cmdline_conf = NULL;   /* global/host configuration values given at the command line */
+
+
+static int config_compare(const config_t *a, const config_t *b) {
+       int result;
+
+       result = strcasecmp(a->variable, b->variable);
+
+       if(result)
+               return result;
+
+       /* give priority to command line options */
+       result = !b->file - !a->file;
+       if (result)
+               return result;
+
+       result = a->line - b->line;
+
+       if(result)
+               return result;
+       else
+               return a->file ? strcmp(a->file, b->file) : 0;
 }
 
-void init_configuration(avl_tree_t **config_tree)
-{
-cp
-  *config_tree = avl_alloc_tree((avl_compare_t)config_compare, (avl_action_t)free_config);
-cp
+void init_configuration(avl_tree_t ** config_tree) {
+       *config_tree = avl_alloc_tree((avl_compare_t) config_compare, (avl_action_t) free_config);
 }
 
-void exit_configuration(avl_tree_t **config_tree)
-{
-cp
-  avl_delete_tree(*config_tree);
-  *config_tree = NULL;
-cp
+void exit_configuration(avl_tree_t ** config_tree) {
+       avl_delete_tree(*config_tree);
+       *config_tree = NULL;
 }
 
-config_t *new_config(void)
-{
-  config_t *cfg;
-cp
-  cfg = (config_t *)xmalloc_and_zero(sizeof(*cfg));
-  
-  return cfg;
+config_t *new_config(void) {
+       return xmalloc_and_zero(sizeof(config_t));
 }
 
-void free_config(config_t *cfg)
-{
-cp
-  if(cfg->variable)
-    free(cfg->variable);
-  if(cfg->value)
-    free(cfg->value);
-  if(cfg->file)
-    free(cfg->file);
-  free(cfg);
-cp
+void free_config(config_t *cfg) {
+       if(cfg->variable)
+               free(cfg->variable);
+
+       if(cfg->value)
+               free(cfg->value);
+
+       if(cfg->file)
+               free(cfg->file);
+
+       free(cfg);
 }
 
-void config_add(avl_tree_t *config_tree, config_t *cfg)
-{
-cp
-  avl_insert(config_tree, cfg);
-cp
+void config_add(avl_tree_t *config_tree, config_t *cfg) {
+       avl_insert(config_tree, cfg);
 }
 
-config_t *lookup_config(avl_tree_t *config_tree, char *variable)
-{
-  config_t cfg, *found;
-cp
-  cfg.variable = variable;
-  cfg.file = "";
-  cfg.line = 0;
+config_t *lookup_config(const avl_tree_t *config_tree, char *variable) {
+       config_t cfg, *found;
+
+       cfg.variable = variable;
+       cfg.file = NULL;
+       cfg.line = 0;
 
-  found = avl_search_closest_greater(config_tree, &cfg);
+       found = avl_search_closest_greater(config_tree, &cfg);
 
-  if(!found)
-    return NULL;
-  
-  if(strcmp(found->variable, variable))
-    return NULL;
+       if(!found)
+               return NULL;
 
-  return found;
+       if(strcasecmp(found->variable, variable))
+               return NULL;
+
+       return found;
 }
 
-config_t *lookup_config_next(avl_tree_t *config_tree, config_t *cfg)
-{
-  avl_node_t *node;
-  config_t *found;
-cp
-  node = avl_search_node(config_tree, cfg);
-  
-  if(node)
-    {
-      if(node->next)
-        {
-          found = (config_t *)node->next->data;
-          if(!strcmp(found->variable, cfg->variable))
-            return found;
-        }
-    }
-  
-  return NULL;
+config_t *lookup_config_next(const avl_tree_t *config_tree, const config_t *cfg) {
+       avl_node_t *node;
+       config_t *found;
+
+       node = avl_search_node(config_tree, cfg);
+
+       if(node) {
+               if(node->next) {
+                       found = node->next->data;
+
+                       if(!strcasecmp(found->variable, cfg->variable))
+                               return found;
+               }
+       }
+
+       return NULL;
 }
-  
-int get_config_bool(config_t *cfg, int *result)
-{
-cp
-  if(!cfg)
-    return 0;
-
-  if(!strcasecmp(cfg->value, "yes"))
-    {
-      *result = 1;
-      return 1;
-    }
-  else if(!strcasecmp(cfg->value, "np"))
-    {
-      *result = 0;
-      return 1;
-    }
-
-  syslog(LOG_ERR, _("\"yes\" or \"no\" expected for configuration variable %s in %s line %d"),
-         cfg->variable, cfg->file, cfg->line);
-
-  return 0;
+
+bool get_config_bool(const config_t *cfg, bool *result) {
+       if(!cfg)
+               return false;
+
+       if(!strcasecmp(cfg->value, "yes")) {
+               *result = true;
+               return true;
+       } else if(!strcasecmp(cfg->value, "no")) {
+               *result = false;
+               return true;
+       }
+
+       logger(LOG_ERR, "\"yes\" or \"no\" expected for configuration variable %s in %s line %d",
+                  cfg->variable, cfg->file, cfg->line);
+
+       return false;
 }
 
-int get_config_int(config_t *cfg, int *result)
-{
-cp
-  if(!cfg)
-    return 0;
-
-  if(sscanf(cfg->value, "%d", result) == 1)
-    return 1;
-    
-  syslog(LOG_ERR, _("Integer expected for configuration variable %s in %s line %d"),
-         cfg->variable, cfg->file, cfg->line);
-  return 0;
+bool get_config_int(const config_t *cfg, int *result) {
+       if(!cfg)
+               return false;
+
+       if(sscanf(cfg->value, "%d", result) == 1)
+               return true;
+
+       logger(LOG_ERR, "Integer expected for configuration variable %s in %s line %d",
+                  cfg->variable, cfg->file, cfg->line);
+
+       return false;
 }
 
-int get_config_string(config_t *cfg, char **result)
-{
-cp
-  if(!cfg)
-    return 0;
+bool get_config_string(const config_t *cfg, char **result) {
+       if(!cfg)
+               return false;
+
+       *result = xstrdup(cfg->value);
 
-  *result = cfg->value;
-  return 1;
+       return true;
 }
 
-int get_config_subnet(config_t *cfg, subnet_t **result)
-{
-  ip_mask_t *ip;
-  subnet_t *subnet;
-cp
-  if(!cfg)
-    return 0;
-
-  ip = strtoip(cfg->value);
-
-  if(!ip)
-    {
-      syslog(LOG_ERR, _("IP address expected for configuration variable %s in %s line %d"),
-             cfg->variable, cfg->file, cfg->line);
-      return 0;
-    }
-  
-  /* Teach newbies what subnets are... */
-
-  if((ip->address & ip->mask) != ip->address)
-    {
-      syslog(LOG_ERR, _("Network address and subnet mask for configuration variable %s in %s line %d"),
-             cfg->variable, cfg->file, cfg->line);
-      free(ip);
-      return 0;
-    }
-
-  subnet = new_subnet();
-  subnet->type = SUBNET_IPV4;
-  subnet->net.ipv4.address = ip->address;
-  subnet->net.ipv4.mask = ip->mask;
-  
-  free(ip);
-
-  *result = subnet;
-  
-  return 1;
+bool get_config_address(const config_t *cfg, struct addrinfo **result) {
+       struct addrinfo *ai;
+
+       if(!cfg)
+               return false;
+
+       ai = str2addrinfo(cfg->value, NULL, 0);
+
+       if(ai) {
+               *result = ai;
+               return true;
+       }
+
+       logger(LOG_ERR, "Hostname or IP address expected for configuration variable %s in %s line %d",
+                  cfg->variable, cfg->file, cfg->line);
+
+       return false;
+}
+
+bool get_config_subnet(const config_t *cfg, subnet_t ** result) {
+       subnet_t subnet = {NULL};
+
+       if(!cfg)
+               return false;
+
+       if(!str2net(&subnet, cfg->value)) {
+               logger(LOG_ERR, "Subnet expected for configuration variable %s in %s line %d",
+                          cfg->variable, cfg->file, cfg->line);
+               return false;
+       }
+
+       /* Teach newbies what subnets are... */
+
+       if(((subnet.type == SUBNET_IPV4)
+               && !maskcheck(&subnet.net.ipv4.address, subnet.net.ipv4.prefixlength, sizeof(ipv4_t)))
+               || ((subnet.type == SUBNET_IPV6)
+               && !maskcheck(&subnet.net.ipv6.address, subnet.net.ipv6.prefixlength, sizeof(ipv6_t)))) {
+               logger(LOG_ERR, "Network address and prefix length do not match for configuration variable %s in %s line %d",
+                          cfg->variable, cfg->file, cfg->line);
+               return false;
+       }
+
+       *(*result = new_subnet()) = subnet;
+
+       return true;
 }
 
 /*
-  Read exactly one line and strip the trailing newline if any.  If the
-  file was on EOF, return NULL. Otherwise, return all the data in a
-  dynamically allocated buffer.
-  
-  If line is non-NULL, it will be used as an initial buffer, to avoid
-  unnecessary mallocing each time this function is called.  If buf is
-  given, and buf needs to be expanded, the var pointed to by buflen
-  will be increased.
+  Read exactly one line and strip the trailing newline if any.
 */
-char *readline(FILE *fp, char **buf, size_t *buflen)
-{
-  char *newline = NULL;
-  char *p;
-  char *line; /* The array that contains everything that has been read
-                 so far */
-  char *idx; /* Read into this pointer, which points to an offset
-                within line */
-  size_t size, newsize; /* The size of the current array pointed to by
-                           line */
-  size_t maxlen; /* Maximum number of characters that may be read with
-                    fgets.  This is newsize - oldsize. */
-
-  if(feof(fp))
-    return NULL;
-
-  if((buf != NULL) && (buflen != NULL))
-    {
-      size = *buflen;
-      line = *buf;
-    }
-  else
-    {
-      size = 100;
-      line = xmalloc(size);
-    }
-
-  maxlen = size;
-  idx = line;
-  *idx = 0;
-  for(;;)
-    {
-      errno = 0;
-      p = fgets(idx, maxlen, fp);
-      if(p == NULL)  /* EOF or error */
-       {
-         if(feof(fp))
-           break;
-
-         /* otherwise: error; let the calling function print an error
-             message if applicable */
-         free(line);
-         return NULL;
-       }
+static char *readline(FILE * fp, char *buf, size_t buflen) {
+       char *newline = NULL;
+       char *p;
+
+       if(feof(fp))
+               return NULL;
+
+       p = fgets(buf, buflen, fp);
 
-      newline = strchr(p, '\n');
-      if(newline == NULL)
-       /* We haven't yet read everything to the end of the line */
-       {
-         newsize = size << 1;
-         line = xrealloc(line, newsize);
-         idx = &line[size - 1];
-         maxlen = newsize - size + 1;
-         size = newsize;
+       if(!p)
+               return NULL;
+
+       newline = strchr(p, '\n');
+
+       if(!newline)
+               return buf;
+
+       *newline = '\0';        /* kill newline */
+       if(newline > p && newline[-1] == '\r')  /* and carriage return if necessary */
+               newline[-1] = '\0';
+
+       return buf;
+}
+
+config_t *parse_config_line(char *line, const char *fname, int lineno) {
+       config_t *cfg;
+       int len;
+       char *variable, *value, *eol;
+       variable = value = line;
+
+       eol = line + strlen(line);
+       while(strchr("\t ", *--eol))
+               *eol = '\0';
+
+       len = strcspn(value, "\t =");
+       value += len;
+       value += strspn(value, "\t ");
+       if(*value == '=') {
+               value++;
+               value += strspn(value, "\t ");
        }
-      else
-       {
-         *newline = '\0'; /* kill newline */
-         break;  /* yay */
+       variable[len] = '\0';
+
+       if(!*value) {
+               const char err[] = "No value for variable";
+               if (fname)
+                       logger(LOG_ERR, "%s `%s' on line %d while reading config file %s",
+                               err, variable, lineno, fname);
+               else
+                       logger(LOG_ERR, "%s `%s' in command line option %d",
+                               err, variable, lineno);
+               return NULL;
        }
-    }
-
-  if((buf != NULL) && (buflen != NULL))
-    {
-      *buflen = size;
-      *buf = line;
-    }
-  return line;
+
+       cfg = new_config();
+       cfg->variable = xstrdup(variable);
+       cfg->value = xstrdup(value);
+       cfg->file = fname ? xstrdup(fname) : NULL;
+       cfg->line = lineno;
+
+       return cfg;
 }
 
 /*
   Parse a configuration file and put the results in the configuration tree
   starting at *base.
 */
-int read_config_file(avl_tree_t *config_tree, const char *fname)
-{
-  int err = -2; /* Parse error */
-  FILE *fp;
-  char *buffer, *line;
-  char *variable, *value;
-  int lineno = 0, ignore = 0;
-  config_t *cfg;
-  size_t bufsize;
-  
-cp
-  if((fp = fopen (fname, "r")) == NULL)
-    {
-      syslog(LOG_ERR, _("Cannot open config file %s: %m"), fname);
-      return -3;
-    }
-
-  bufsize = 100;
-  buffer = xmalloc(bufsize);
-  
-  for(;;)
-    {
-      if((line = readline(fp, &buffer, &bufsize)) == NULL)
-       {
-         err = -1;
-         break;
+bool read_config_file(avl_tree_t *config_tree, const char *fname) {
+       FILE *fp;
+       char buffer[MAX_STRING_SIZE];
+       char *line;
+       int lineno = 0;
+       bool ignore = false;
+       config_t *cfg;
+       bool result = false;
+
+       fp = fopen(fname, "r");
+
+       if(!fp) {
+               logger(LOG_ERR, "Cannot open config file %s: %s", fname, strerror(errno));
+               return false;
        }
 
-      if(feof(fp))
-       {
-         err = 0;
-         break;
+       for(;;) {
+               line = readline(fp, buffer, sizeof buffer);
+
+               if(!line) {
+                       if(feof(fp))
+                               result = true;
+                       break;
+               }
+
+               lineno++;
+
+               if(!*line || *line == '#')
+                       continue;
+
+               if(ignore) {
+                       if(!strncmp(line, "-----END", 8))
+                               ignore = false;
+                       continue;
+               }
+               
+               if(!strncmp(line, "-----BEGIN", 10)) {
+                       ignore = true;
+                       continue;
+               }
+
+               cfg = parse_config_line(line, fname, lineno);
+               if (!cfg)
+                       break;
+               config_add(config_tree, cfg);
        }
 
-      lineno++;
-
-      if((variable = strtok(line, "\t =")) == NULL)
-       continue; /* no tokens on this line */
-
-      if(variable[0] == '#')
-       continue; /* comment: ignore */
-
-      if(!strcmp(variable, "-----BEGIN"))
-        ignore = 1;
-        
-      if(!ignore)
-        {
-          if(((value = strtok(NULL, "\t\n\r =")) == NULL) || value[0] == '#')
-           {
-             syslog(LOG_ERR, _("No value for variable `%s' on line %d while reading config file %s"),
-                     variable, lineno, fname);
-             break;
-           }
-
-          cfg = new_config();
-          cfg->variable = xstrdup(variable);
-          cfg->value = xstrdup(value);
-          cfg->file = xstrdup(fname);
-          cfg->line = lineno;
-
-          config_add(config_tree, cfg);
-       }
-
-      if(!strcmp(variable, "-----END"))
-        ignore = 0;
-    }
-
-  free(buffer);
-  fclose (fp);
-cp
-  return err;
+       fclose(fp);
+
+       return result;
 }
 
-int read_server_config()
-{
-  char *fname;
-  int x;
-cp
-  asprintf(&fname, "%s/tinc.conf", confbase);
-  x = read_config_file(config_tree, fname);
-  if(x == -1) /* System error: complain */
-    {
-      syslog(LOG_ERR, _("Failed to read `%s': %m"),
-             fname);
-    }
-  free(fname);
-cp
-  return x;  
+void read_config_options(avl_tree_t *config_tree, const char *prefix) {
+       list_node_t *node, *next;
+       size_t prefix_len = prefix ? strlen(prefix) : 0;
+
+       for(node = cmdline_conf->tail; node; node = next) {
+               config_t *orig_cfg, *cfg = (config_t *)node->data;
+               next = node->prev;
+
+               if(!prefix) {
+                       if(strchr(cfg->variable, '.'))
+                               continue;
+                       node->data = NULL;
+                       list_unlink_node(cmdline_conf, node);
+               } else {
+                       if(strncmp(prefix, cfg->variable, prefix_len) ||
+                          cfg->variable[prefix_len] != '.')
+                               continue;
+                       /* Because host configuration is parsed again when
+                          reconnecting, nodes must not be freed when a prefix
+                          is given. */
+                       orig_cfg = cfg;
+                       cfg = new_config();
+                       cfg->variable = xstrdup(orig_cfg->variable + prefix_len + 1);
+                       cfg->value = xstrdup(orig_cfg->value);
+                       cfg->file = NULL;
+                       cfg->line = orig_cfg->line;
+               }
+               config_add(config_tree, cfg);
+       }
 }
 
-int isadir(const char* f)
-{
-  struct stat s;
+bool read_server_config(void) {
+       char *fname;
+       bool x;
 
-  if(stat(f, &s) < 0)
-    return 0;
-  else
-    return S_ISDIR(s.st_mode);
+       read_config_options(config_tree, NULL);
+
+       xasprintf(&fname, "%s/tinc.conf", confbase);
+       x = read_config_file(config_tree, fname);
+
+       if(!x) {                                /* System error: complain */
+               logger(LOG_ERR, "Failed to read `%s': %s", fname, strerror(errno));
+       }
+
+       free(fname);
+
+       return x;
+}
+
+bool read_connection_config(connection_t *c) {
+       char *fname;
+       bool x;
+
+       read_config_options(c->config_tree, c->name);
+
+       xasprintf(&fname, "%s/hosts/%s", confbase, c->name);
+       x = read_config_file(c->config_tree, fname);
+       free(fname);
+
+       return x;
 }
 
-int is_safe_path(const char *file)
-{
-  char *p;
-  const char *f;
-  char x;
-  struct stat s;
-  char l[MAXBUFSIZE];
-
-  if(*file != '/')
-    {
-      syslog(LOG_ERR, _("`%s' is not an absolute path"), file);
-      return 0;
-    }
-
-  p = strrchr(file, '/');
-  
-  if(p == file)                /* It's in the root */
-    p++;
-    
-  x = *p;
-  *p = '\0';
-
-  f = file;
-check1:
-  if(lstat(f, &s) < 0)
-    {
-      syslog(LOG_ERR, _("Couldn't stat `%s': %m"),
-             f);
-      return 0;
-    }
-
-  if(s.st_uid != geteuid())
-    {
-      syslog(LOG_ERR, _("`%s' is owned by UID %d instead of %d"),
-             f, s.st_uid, geteuid());
-      return 0;
-    }
-
-  if(S_ISLNK(s.st_mode))
-    {
-      syslog(LOG_WARNING, _("Warning: `%s' is a symlink"),
-             f);
-
-      if(readlink(f, l, MAXBUFSIZE) < 0)
-        {
-          syslog(LOG_ERR, _("Unable to read symbolic link `%s': %m"), f);
-          return 0;
-        }
-      
-      f = l;
-      goto check1;
-    }
-
-  *p = x;
-  f = file;
-  
-check2:
-  if(lstat(f, &s) < 0 && errno != ENOENT)
-    {
-      syslog(LOG_ERR, _("Couldn't stat `%s': %m"),
-             f);
-      return 0;
-    }
-    
-  if(errno == ENOENT)
-    return 1;
-
-  if(s.st_uid != geteuid())
-    {
-      syslog(LOG_ERR, _("`%s' is owned by UID %d instead of %d"),
-             f, s.st_uid, geteuid());
-      return 0;
-    }
-
-  if(S_ISLNK(s.st_mode))
-    {
-      syslog(LOG_WARNING, _("Warning: `%s' is a symlink"),
-             f);
-
-      if(readlink(f, l, MAXBUFSIZE) < 0)
-        {
-          syslog(LOG_ERR, _("Unable to read symbolic link `%s': %m"), f);
-          return 0;
-        }
-      
-      f = l;
-      goto check2;
-    }
-
-  if(s.st_mode & 0007)
-    {
-      /* Accessible by others */
-      syslog(LOG_ERR, _("`%s' has unsecure permissions"),
-             f);
-      return 0;
-    }
-  
-  return 1;
+FILE *ask_and_open(const char *filename, const char *what) {
+       FILE *r;
+       char *directory;
+       char line[PATH_MAX];
+       const char *fn;
+
+       /* Check stdin and stdout */
+       if(!isatty(0) || !isatty(1)) {
+               /* Argh, they are running us from a script or something.  Write
+                  the files to the current directory and let them burn in hell
+                  for ever. */
+               fn = filename;
+       } else {
+               /* Ask for a file and/or directory name. */
+               fprintf(stdout, "Please enter a file to save %s to [%s]: ",
+                               what, filename);
+               fflush(stdout);
+
+               fn = readline(stdin, line, sizeof line);
+
+               if(!fn) {
+                       fprintf(stderr, "Error while reading stdin: %s\n",
+                                       strerror(errno));
+                       return NULL;
+               }
+
+               if(!strlen(fn))
+                       /* User just pressed enter. */
+                       fn = filename;
+       }
+
+#ifdef HAVE_MINGW
+       if(fn[0] != '\\' && fn[0] != '/' && !strchr(fn, ':')) {
+#else
+       if(fn[0] != '/') {
+#endif
+               /* The directory is a relative path or a filename. */
+               char *p;
+
+               directory = get_current_dir_name();
+               xasprintf(&p, "%s/%s", directory, fn);
+               free(directory);
+               fn = p;
+       }
+
+       umask(0077);                            /* Disallow everything for group and other */
+
+       /* Open it first to keep the inode busy */
+
+       r = fopen(fn, "r+") ?: fopen(fn, "w+");
+
+       if(!r) {
+               fprintf(stderr, "Error opening file `%s': %s\n",
+                               fn, strerror(errno));
+               return NULL;
+       }
+
+       return r;
 }
 
-FILE *ask_and_safe_open(const char* filename, const char* what, const char* mode)
-{
-  FILE *r;
-  char *directory;
-  char *fn;
-
-  /* Check stdin and stdout */
-  if(!isatty(0) || !isatty(1))
-    {
-      /* Argh, they are running us from a script or something.  Write
-         the files to the current directory and let them burn in hell
-         for ever. */
-      fn = xstrdup(filename);
-    }
-  else
-    {
-      /* Ask for a file and/or directory name. */
-      fprintf(stdout, _("Please enter a file to save %s to [%s]: "),
-             what, filename);
-      fflush(stdout);
-
-      if((fn = readline(stdin, NULL, NULL)) == NULL)
-       {
-         fprintf(stderr, _("Error while reading stdin: %s\n"), strerror(errno));
-         return NULL;
+bool disable_old_keys(FILE *f) {
+       char buf[100];
+       long pos;
+       bool disabled = false;
+
+       rewind(f);
+       pos = ftell(f);
+
+       if(pos < 0)
+               return false;
+
+       while(fgets(buf, sizeof buf, f)) {
+               if(!strncmp(buf, "-----BEGIN RSA", 14)) {       
+                       buf[11] = 'O';
+                       buf[12] = 'L';
+                       buf[13] = 'D';
+                       if(fseek(f, pos, SEEK_SET))
+                               break;
+                       if(fputs(buf, f) <= 0)
+                               break;
+                       disabled = true;
+               }
+               else if(!strncmp(buf, "-----END RSA", 12)) {    
+                       buf[ 9] = 'O';
+                       buf[10] = 'L';
+                       buf[11] = 'D';
+                       if(fseek(f, pos, SEEK_SET))
+                               break;
+                       if(fputs(buf, f) <= 0)
+                               break;
+                       disabled = true;
+               }
+               pos = ftell(f);
+               if(pos < 0)
+                       break;
        }
 
-      if(strlen(fn) == 0)
-       /* User just pressed enter. */
-       fn = xstrdup(filename);
-    }
-
-  if((strchr(fn, '/') == NULL) || (fn[0] != '/'))
-    {
-      /* The directory is a relative path or a filename. */
-      char *p;
-      
-      directory = get_current_dir_name();
-      asprintf(&p, "%s/%s", directory, fn);
-      free(fn);
-      free(directory);
-      fn = p;
-    }
-
-  umask(0077); /* Disallow everything for group and other */
-  
-  /* Open it first to keep the inode busy */
-  if((r = fopen(fn, mode)) == NULL)
-    {
-      fprintf(stderr, _("Error opening file `%s': %s\n"),
-             fn, strerror(errno));
-      free(fn);
-      return NULL;
-    }
-    
-  /* Then check the file for nasty attacks */
-  if(!is_safe_path(fn))  /* Do not permit any directories that are
-                            readable or writeable by other users. */
-    {
-      fprintf(stderr, _("The file `%s' (or any of the leading directories) has unsafe permissions.\n"
-                       "I will not create or overwrite this file.\n"),
-                       fn);
-      fclose(r);
-      free(fn);
-      return NULL;
-    }
-
-  free(fn);
-
-  return r;
+       return disabled;
 }