Browse Source

don't bother checking impossible condition

the config readear already verified that at least host or tunnel are
set.
1.1
Oswald Buddenhagen 11 years ago
parent
commit
f55f42bdfc
  1. 5
      src/compat/config.c

5
src/compat/config.c

@ -245,7 +245,7 @@ write_imap_server( FILE *fp, config_t *cfg )
if (cfg->tunnel) if (cfg->tunnel)
nfasprintf( (char **)&cfg->old_server_name, "tunnel%d", ++tunnels ); nfasprintf( (char **)&cfg->old_server_name, "tunnel%d", ++tunnels );
else if (cfg->host) { else {
if (sscanf( cfg->host, "%d.%d.%d.%d", &a1, &a2, &a3, &a4 ) == 4) if (sscanf( cfg->host, "%d.%d.%d.%d", &a1, &a2, &a3, &a4 ) == 4)
hl = nfsnprintf( buf, sizeof(buf), "%s", cfg->host ); hl = nfsnprintf( buf, sizeof(buf), "%s", cfg->host );
else { else {
@ -269,9 +269,6 @@ write_imap_server( FILE *fp, config_t *cfg )
cfg->old_server_name = nfstrdup( buf ); cfg->old_server_name = nfstrdup( buf );
cfg->old_servers = 1; cfg->old_servers = 1;
gotsrv: ; gotsrv: ;
} else {
fprintf( stderr, "ERROR: Neither host nor tunnel specified for mailbox %s.\n", cfg->path );
exit( 1 );
} }
if (cfg->user) if (cfg->user)

Loading…
Cancel
Save