[authfile.c]
     spelling.  use sizeof vs. strlen().  ok markus@
This commit is contained in:
Ben Lindstrom 2001-01-29 07:51:26 +00:00
parent 36579d3daa
commit 1170d71b5d
2 changed files with 16 additions and 13 deletions

View File

@ -9,6 +9,9 @@
ssh-dss.h ssh-keygen.1 ssh-keyscan.1 ssh-rsa.h ssh1.h ssh_config ssh-dss.h ssh-keygen.1 ssh-keyscan.1 ssh-rsa.h ssh1.h ssh_config
sshconnect.h sshd_config tildexpand.h uidswap.h uuencode.h] sshconnect.h sshd_config tildexpand.h uidswap.h uuencode.h]
$OpenBSD$ $OpenBSD$
- stevesk@cvs.openbsd.org 2001/01/28 22:27:05
[authfile.c]
spelling. use sizeof vs. strlen(). ok markus@
20010126 20010126
- (bal) SSH_PROGRAM vs _PATH_SSH_PROGRAM fix pointed out by Roumen - (bal) SSH_PROGRAM vs _PATH_SSH_PROGRAM fix pointed out by Roumen

View File

@ -36,7 +36,7 @@
*/ */
#include "includes.h" #include "includes.h"
RCSID("$OpenBSD: authfile.c,v 1.25 2001/01/21 19:05:44 markus Exp $"); RCSID("$OpenBSD: authfile.c,v 1.26 2001/01/28 22:27:05 stevesk Exp $");
#include <openssl/err.h> #include <openssl/err.h>
#include <openssl/evp.h> #include <openssl/evp.h>
@ -51,7 +51,8 @@ RCSID("$OpenBSD: authfile.c,v 1.25 2001/01/21 19:05:44 markus Exp $");
#include "log.h" #include "log.h"
/* Version identification string for identity files. */ /* Version identification string for identity files. */
#define AUTHFILE_ID_STRING "SSH PRIVATE KEY FILE FORMAT 1.1\n" static const char authfile_id_string[] =
"SSH PRIVATE KEY FILE FORMAT 1.1\n";
/* /*
* Saves the authentication (private) key in a file, encrypting it with * Saves the authentication (private) key in a file, encrypting it with
@ -111,9 +112,8 @@ save_private_key_rsa1(const char *filename, const char *passphrase,
buffer_init(&encrypted); buffer_init(&encrypted);
/* First store keyfile id string. */ /* First store keyfile id string. */
cp = AUTHFILE_ID_STRING; for (i = 0; authfile_id_string[i]; i++)
for (i = 0; cp[i]; i++) buffer_put_char(&encrypted, authfile_id_string[i]);
buffer_put_char(&encrypted, cp[i]);
buffer_put_char(&encrypted, 0); buffer_put_char(&encrypted, 0);
/* Store cipher type. */ /* Store cipher type. */
@ -247,8 +247,8 @@ load_public_key_rsa(const char *filename, RSA * pub, char **comment_return)
} }
close(fd); close(fd);
/* Check that it is at least big enought to contain the ID string. */ /* Check that it is at least big enough to contain the ID string. */
if (len < strlen(AUTHFILE_ID_STRING) + 1) { if (len < sizeof(authfile_id_string)) {
debug3("Bad RSA1 key file %.200s.", filename); debug3("Bad RSA1 key file %.200s.", filename);
buffer_free(&buffer); buffer_free(&buffer);
return 0; return 0;
@ -257,8 +257,8 @@ load_public_key_rsa(const char *filename, RSA * pub, char **comment_return)
* Make sure it begins with the id string. Consume the id string * Make sure it begins with the id string. Consume the id string
* from the buffer. * from the buffer.
*/ */
for (i = 0; i < (u_int) strlen(AUTHFILE_ID_STRING) + 1; i++) for (i = 0; i < sizeof(authfile_id_string); i++)
if (buffer_get_char(&buffer) != (u_char) AUTHFILE_ID_STRING[i]) { if (buffer_get_char(&buffer) != authfile_id_string[i]) {
debug3("Bad RSA1 key file %.200s.", filename); debug3("Bad RSA1 key file %.200s.", filename);
buffer_free(&buffer); buffer_free(&buffer);
return 0; return 0;
@ -337,8 +337,8 @@ load_private_key_rsa1(int fd, const char *filename,
} }
close(fd); close(fd);
/* Check that it is at least big enought to contain the ID string. */ /* Check that it is at least big enough to contain the ID string. */
if (len < strlen(AUTHFILE_ID_STRING) + 1) { if (len < sizeof(authfile_id_string)) {
debug3("Bad RSA1 key file %.200s.", filename); debug3("Bad RSA1 key file %.200s.", filename);
buffer_free(&buffer); buffer_free(&buffer);
return 0; return 0;
@ -347,8 +347,8 @@ load_private_key_rsa1(int fd, const char *filename,
* Make sure it begins with the id string. Consume the id string * Make sure it begins with the id string. Consume the id string
* from the buffer. * from the buffer.
*/ */
for (i = 0; i < (u_int) strlen(AUTHFILE_ID_STRING) + 1; i++) for (i = 0; i < sizeof(authfile_id_string); i++)
if (buffer_get_char(&buffer) != (u_char) AUTHFILE_ID_STRING[i]) { if (buffer_get_char(&buffer) != authfile_id_string[i]) {
debug3("Bad RSA1 key file %.200s.", filename); debug3("Bad RSA1 key file %.200s.", filename);
buffer_free(&buffer); buffer_free(&buffer);
return 0; return 0;