diff options
| author | djm <djm> | 2005-06-17 02:59:34 +0000 |
|---|---|---|
| committer | djm <djm> | 2005-06-17 02:59:34 +0000 |
| commit | a76a07381c5d7d984011e9a9721d7f0c9c6e9ff3 (patch) | |
| tree | 9ec2978d20b927f5b9f87e80505f7f806e009b97 /bufaux.c | |
| parent | fcd3b6d9548efdf5eee9020f8b01337677a986eb (diff) | |
| download | openssh-a76a07381c5d7d984011e9a9721d7f0c9c6e9ff3.tar.gz | |
- djm@cvs.openbsd.org 2005/06/17 02:44:33
[auth-rsa.c auth.c auth1.c auth2-chall.c auth2-gss.c authfd.c authfile.c]
[bufaux.c canohost.c channels.c cipher.c clientloop.c dns.c gss-serv.c]
[kex.c kex.h key.c mac.c match.c misc.c packet.c packet.h scp.c]
[servconf.c session.c session.h sftp-client.c sftp-server.c sftp.c]
[ssh-keyscan.c ssh-rsa.c sshconnect.c sshconnect1.c sshconnect2.c sshd.c]
make this -Wsign-compare clean; ok avsm@ markus@
NB. auth1.c changes not committed yet (conflicts with uncommitted sync)
NB2. more work may be needed to make portable Wsign-compare clean
Diffstat (limited to 'bufaux.c')
| -rw-r--r-- | bufaux.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -37,7 +37,7 @@ */ #include "includes.h" -RCSID("$OpenBSD: bufaux.c,v 1.35 2005/03/10 22:01:05 deraadt Exp $"); +RCSID("$OpenBSD: bufaux.c,v 1.36 2005/06/17 02:44:32 djm Exp $"); #include <openssl/bn.h> #include "bufaux.h" @@ -154,7 +154,7 @@ buffer_put_bignum2_ret(Buffer *buffer, const BIGNUM *value) buf[0] = 0x00; /* Get the value of in binary */ oi = BN_bn2bin(value, buf+1); - if (oi != bytes-1) { + if (oi < 0 || (u_int)oi != bytes - 1) { error("buffer_put_bignum2_ret: BN_bn2bin() failed: " "oi %d != bin_size %d", oi, bytes); xfree(buf); |