diff options
| author |   djm <djm@openbsd.org> | 2009-01-22 10:02:34 +0000 |
|---|---|---|
| committer | djm <djm@openbsd.org> | 2009-01-22 10:02:34 +0000 |
| commit | a10ba8e0d42f8de2c7e5b81cac0336819d636310 (patch) | |
| tree | cf85b99b29b096fb5939ba93f194d90b5b10dd35 /usr.bin/ssh/ssh-keyscan.c | |
| parent | oops! I committed the wrong version of the Channel->path diff, (diff) | |
| download | wireguard-openbsd-a10ba8e0d42f8de2c7e5b81cac0336819d636310.tar.xz wireguard-openbsd-a10ba8e0d42f8de2c7e5b81cac0336819d636310.zip | |
make a2port() return -1 when it encounters an invalid port number
rather than 0, which it will now treat as valid (needed for future work)
adjust current consumers of a2port() to check its return value is <= 0,
which in turn required some things to be converted from u_short => int
make use of int vs. u_short consistent in some other places too
feedback & ok markus@
Diffstat (limited to 'usr.bin/ssh/ssh-keyscan.c')
| -rw-r--r-- | usr.bin/ssh/ssh-keyscan.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.bin/ssh/ssh-keyscan.c b/usr.bin/ssh/ssh-keyscan.c index 40a9966b17c..2edfc29db3f 100644 --- a/usr.bin/ssh/ssh-keyscan.c +++ b/usr.bin/ssh/ssh-keyscan.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ssh-keyscan.c,v 1.77 2008/11/01 11:14:36 sobrado Exp $ */ +/* $OpenBSD: ssh-keyscan.c,v 1.78 2009/01/22 10:02:34 djm Exp $ */ /* * Copyright 1995, 1996 by David Mazieres <dm@lcs.mit.edu>. * @@ -730,7 +730,7 @@ main(int argc, char **argv) break; case 'p': ssh_port = a2port(optarg); - if (ssh_port == 0) { + if (ssh_port <= 0) { fprintf(stderr, "Bad port '%s'\n", optarg); exit(1); } |