revert __bounded change; it causes way more problems for portable than

it solves; pointed out by dtucker@

diff --git a/usr.bin/ssh/chacha.h b/usr.bin/ssh/chacha.h
index ea57fe1..40eaf2d 100644 (file)
--- a/usr.bin/ssh/chacha.h
+++ b/usr.bin/ssh/chacha.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: chacha.h,v 1.2 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: chacha.h,v 1.3 2014/05/02 03:27:54 djm Exp $ */
 
 /*
 chacha-merged.c version 20080118
@@ -22,14 +22,14 @@ struct chacha_ctx {
 #define CHACHA_BLOCKLEN                64
 
 void chacha_keysetup(struct chacha_ctx *x, const u_char *k, u_int kbits)
-    __bounded((__minbytes__, 2, CHACHA_MINKEYLEN));
+    __attribute__((__bounded__(__minbytes__, 2, CHACHA_MINKEYLEN)));
 void chacha_ivsetup(struct chacha_ctx *x, const u_char *iv, const u_char *ctr)
-    __bounded((__minbytes__, 2, CHACHA_NONCELEN))
-    __bounded((__minbytes__, 3, CHACHA_CTRLEN));
+    __attribute__((__bounded__(__minbytes__, 2, CHACHA_NONCELEN)))
+    __attribute__((__bounded__(__minbytes__, 3, CHACHA_CTRLEN)));
 void chacha_encrypt_bytes(struct chacha_ctx *x, const u_char *m,
     u_char *c, u_int bytes)
-    __bounded((__buffer__, 2, 4))
-    __bounded((__buffer__, 3, 4));
+    __attribute__((__bounded__(__buffer__, 2, 4)))
+    __attribute__((__bounded__(__buffer__, 3, 4)));
 
 #endif /* CHACHA_H */
 

diff --git a/usr.bin/ssh/cipher-chachapoly.h b/usr.bin/ssh/cipher-chachapoly.h
index 35326b9..7948dcd 100644 (file)
--- a/usr.bin/ssh/cipher-chachapoly.h
+++ b/usr.bin/ssh/cipher-chachapoly.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: cipher-chachapoly.h,v 1.2 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: cipher-chachapoly.h,v 1.3 2014/05/02 03:27:54 djm Exp $ */
 
 /*
  * Copyright (c) Damien Miller 2013 <djm@mindrot.org>
@@ -30,12 +30,12 @@ struct chachapoly_ctx {
 
 void   chachapoly_init(struct chachapoly_ctx *cpctx,
     const u_char *key, u_int keylen)
-    __bounded((__buffer__, 2, 3));
+    __attribute__((__bounded__(__buffer__, 2, 3)));
 int    chachapoly_crypt(struct chachapoly_ctx *cpctx, u_int seqnr,
     u_char *dest, const u_char *src, u_int len, u_int aadlen, u_int authlen,
     int do_encrypt);
 int    chachapoly_get_length(struct chachapoly_ctx *cpctx,
     u_int *plenp, u_int seqnr, const u_char *cp, u_int len)
-    __bounded((__buffer__, 4, 5));
+    __attribute__((__bounded__(__buffer__, 4, 5)));
 
 #endif /* CHACHA_POLY_AEAD_H */

diff --git a/usr.bin/ssh/digest.h b/usr.bin/ssh/digest.h
index 0edd0a7..04295e2 100644 (file)
--- a/usr.bin/ssh/digest.h
+++ b/usr.bin/ssh/digest.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: digest.h,v 1.3 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: digest.h,v 1.4 2014/05/02 03:27:54 djm Exp $ */
 /*
  * Copyright (c) 2013 Damien Miller <djm@mindrot.org>
  *
@@ -45,18 +45,18 @@ int ssh_digest_copy_state(struct ssh_digest_ctx *from,
 /* One-shot API */
 int ssh_digest_memory(int alg, const void *m, size_t mlen,
     u_char *d, size_t dlen)
-       __bounded((__buffer__, 2, 3))
-       __bounded((__buffer__, 4, 5));
+       __attribute__((__bounded__(__buffer__, 2, 3)))
+       __attribute__((__bounded__(__buffer__, 4, 5)));
 int ssh_digest_buffer(int alg, const Buffer *b, u_char *d, size_t dlen)
-       __bounded((__buffer__, 3, 4));
+       __attribute__((__bounded__(__buffer__, 3, 4)));
 
 /* Update API */
 struct ssh_digest_ctx *ssh_digest_start(int alg);
 int ssh_digest_update(struct ssh_digest_ctx *ctx, const void *m, size_t mlen)
-       __bounded((__buffer__, 2, 3));
+       __attribute__((__bounded__(__buffer__, 2, 3)));
 int ssh_digest_update_buffer(struct ssh_digest_ctx *ctx, const Buffer *b);
 int ssh_digest_final(struct ssh_digest_ctx *ctx, u_char *d, size_t dlen)
-       __bounded((__buffer__, 2, 3));
+       __attribute__((__bounded__(__buffer__, 2, 3)));
 void ssh_digest_free(struct ssh_digest_ctx *ctx);
 
 #endif /* _DIGEST_H */

diff --git a/usr.bin/ssh/hmac.h b/usr.bin/ssh/hmac.h
index 9eddbe2..0581390 100644 (file)
--- a/usr.bin/ssh/hmac.h
+++ b/usr.bin/ssh/hmac.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: hmac.h,v 1.7 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: hmac.h,v 1.8 2014/05/02 03:27:54 djm Exp $ */
 /*
  * Copyright (c) 2014 Markus Friedl.  All rights reserved.
  *
@@ -26,12 +26,12 @@ struct ssh_hmac_ctx *ssh_hmac_start(int alg);
 
 /* Sets the state of the HMAC or resets the state if key == NULL */
 int ssh_hmac_init(struct ssh_hmac_ctx *ctx, const void *key, size_t klen)
-       __bounded((__buffer__, 2, 3));
+       __attribute__((__bounded__(__buffer__, 2, 3)));
 int ssh_hmac_update(struct ssh_hmac_ctx *ctx, const void *m, size_t mlen)
-       __bounded((__buffer__, 2, 3));
+       __attribute__((__bounded__(__buffer__, 2, 3)));
 int ssh_hmac_update_buffer(struct ssh_hmac_ctx *ctx, const Buffer *b);
 int ssh_hmac_final(struct ssh_hmac_ctx *ctx, u_char *d, size_t dlen)
-       __bounded((__buffer__, 2, 3));
+       __attribute__((__bounded__(__buffer__, 2, 3)));
 void ssh_hmac_free(struct ssh_hmac_ctx *ctx);
 
 #endif /* _HMAC_H */

diff --git a/usr.bin/ssh/kex.h b/usr.bin/ssh/kex.h
index a1c1cff..49cc244 100644 (file)
--- a/usr.bin/ssh/kex.h
+++ b/usr.bin/ssh/kex.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: kex.h,v 1.63 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: kex.h,v 1.64 2014/05/02 03:27:54 djm Exp $ */
 
 /*
  * Copyright (c) 2000, 2001 Markus Friedl.  All rights reserved.
@@ -182,12 +182,12 @@ kex_c25519_hash(int, char *, char *, char *, int,
 
 #define CURVE25519_SIZE 32
 void   kexc25519_keygen(u_char[CURVE25519_SIZE], u_char[CURVE25519_SIZE])
-       __bounded((__minbytes__, 1, CURVE25519_SIZE))
-       __bounded((__minbytes__, 2, CURVE25519_SIZE));
+       __attribute__((__bounded__(__minbytes__, 1, CURVE25519_SIZE)))
+       __attribute__((__bounded__(__minbytes__, 2, CURVE25519_SIZE)));
 void kexc25519_shared_key(const u_char key[CURVE25519_SIZE],
     const u_char pub[CURVE25519_SIZE], Buffer *out)
-       __bounded((__minbytes__, 1, CURVE25519_SIZE))
-       __bounded((__minbytes__, 2, CURVE25519_SIZE));
+       __attribute__((__bounded__(__minbytes__, 1, CURVE25519_SIZE)))
+       __attribute__((__bounded__(__minbytes__, 2, CURVE25519_SIZE)));
 
 void
 derive_ssh1_session_id(BIGNUM *, BIGNUM *, u_int8_t[8], u_int8_t[16]);

diff --git a/usr.bin/ssh/kexc25519.c b/usr.bin/ssh/kexc25519.c
index a8a2b07..93ca2dd 100644 (file)
--- a/usr.bin/ssh/kexc25519.c
+++ b/usr.bin/ssh/kexc25519.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kexc25519.c,v 1.6 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: kexc25519.c,v 1.7 2014/05/02 03:27:54 djm Exp $ */
 /*
  * Copyright (c) 2001, 2013 Markus Friedl.  All rights reserved.
  * Copyright (c) 2010 Damien Miller.  All rights reserved.
@@ -43,9 +43,9 @@
 
 extern int crypto_scalarmult_curve25519(u_char a[CURVE25519_SIZE],
     const u_char b[CURVE25519_SIZE], const u_char c[CURVE25519_SIZE])
-       __bounded((__minbytes__, 1, CURVE25519_SIZE))
-       __bounded((__minbytes__, 2, CURVE25519_SIZE))
-       __bounded((__minbytes__, 3, CURVE25519_SIZE));
+       __attribute__((__bounded__(__minbytes__, 1, CURVE25519_SIZE)))
+       __attribute__((__bounded__(__minbytes__, 2, CURVE25519_SIZE)))
+       __attribute__((__bounded__(__minbytes__, 3, CURVE25519_SIZE)));
 
 void
 kexc25519_keygen(u_char key[CURVE25519_SIZE], u_char pub[CURVE25519_SIZE])

diff --git a/usr.bin/ssh/misc.h b/usr.bin/ssh/misc.h
index 435122e..693fff8 100644 (file)
--- a/usr.bin/ssh/misc.h
+++ b/usr.bin/ssh/misc.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: misc.h,v 1.52 2014/04/20 02:30:25 djm Exp $ */
+/* $OpenBSD: misc.h,v 1.53 2014/05/02 03:27:54 djm Exp $ */
 
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
@@ -68,23 +68,23 @@ int  tun_open(int, int);
 
 /* Functions to extract or store big-endian words of various sizes */
 u_int64_t      get_u64(const void *)
-    __bounded(( __minbytes__, 1, 8));
+    __attribute__((__bounded__( __minbytes__, 1, 8)));
 u_int32_t      get_u32(const void *)
-    __bounded(( __minbytes__, 1, 4));
+    __attribute__((__bounded__( __minbytes__, 1, 4)));
 u_int16_t      get_u16(const void *)
-    __bounded(( __minbytes__, 1, 2));
+    __attribute__((__bounded__( __minbytes__, 1, 2)));
 void           put_u64(void *, u_int64_t)
-    __bounded(( __minbytes__, 1, 8));
+    __attribute__((__bounded__( __minbytes__, 1, 8)));
 void           put_u32(void *, u_int32_t)
-    __bounded(( __minbytes__, 1, 4));
+    __attribute__((__bounded__( __minbytes__, 1, 4)));
 void           put_u16(void *, u_int16_t)
-    __bounded(( __minbytes__, 1, 2));
+    __attribute__((__bounded__( __minbytes__, 1, 2)));
 
 /* Little-endian store/load, used by umac.c */
 u_int32_t      get_u32_le(const void *)
-    __bounded(( __minbytes__, 1, 4));
+    __attribute__((__bounded__(__minbytes__, 1, 4)));
 void           put_u32_le(void *, u_int32_t)
-    __bounded(( __minbytes__, 1, 4));
+    __attribute__((__bounded__(__minbytes__, 1, 4)));
 
 struct bwlimit {
        size_t buflen;

diff --git a/usr.bin/ssh/poly1305.h b/usr.bin/ssh/poly1305.h
index 82373e6..f7db5f8 100644 (file)
--- a/usr.bin/ssh/poly1305.h
+++ b/usr.bin/ssh/poly1305.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: poly1305.h,v 1.3 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: poly1305.h,v 1.4 2014/05/02 03:27:54 djm Exp $ */
 
 /* 
  * Public Domain poly1305 from Andrew Moon
@@ -15,8 +15,8 @@
 
 void poly1305_auth(u_char out[POLY1305_TAGLEN], const u_char *m, size_t inlen,
     const u_char key[POLY1305_KEYLEN])
-    __bounded((__minbytes__, 1, POLY1305_TAGLEN))
-    __bounded((__buffer__, 2, 3))
-    __bounded((__minbytes__, 4, POLY1305_KEYLEN));
+    __attribute__((__bounded__(__minbytes__, 1, POLY1305_TAGLEN)))
+    __attribute__((__bounded__(__buffer__, 2, 3)))
+    __attribute__((__bounded__(__minbytes__, 4, POLY1305_KEYLEN)));
 
 #endif /* POLY1305_H */

diff --git a/usr.bin/ssh/ssh-pkcs11.c b/usr.bin/ssh/ssh-pkcs11.c
index 71b6ab9..fb5bf16 100644 (file)
--- a/usr.bin/ssh/ssh-pkcs11.c
+++ b/usr.bin/ssh/ssh-pkcs11.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-pkcs11.c,v 1.12 2014/03/26 04:55:35 djm Exp $ */
+/* $OpenBSD: ssh-pkcs11.c,v 1.13 2014/05/02 03:27:54 djm Exp $ */
 /*
  * Copyright (c) 2010 Markus Friedl.  All rights reserved.
  *
@@ -377,7 +377,7 @@ pkcs11_open_session(struct pkcs11_provider *p, CK_ULONG slotidx, char *pin)
  */
 static int pkcs11_fetch_keys_filter(struct pkcs11_provider *, CK_ULONG,
     CK_ATTRIBUTE [], CK_ATTRIBUTE [3], Key ***, int *)
-       __bounded((__minbytes__,4, 3 * sizeof(CK_ATTRIBUTE)));
+       __attribute__((__bounded__(__minbytes__,4, 3 * sizeof(CK_ATTRIBUTE))));
 
 static int
 pkcs11_fetch_keys(struct pkcs11_provider *p, CK_ULONG slotidx,