fix for SVN r253208 breaking buildkernel with gcc
Andre Oppermann
andre at freebsd.org
Thu Jul 11 16:28:00 UTC 2013
On 11.07.2013 18:09, Michael Butler wrote:
> On 07/11/13 12:07, Michael Butler wrote:
>> Seems gcc is rather fussy about propagating 'const' and fails to compile
>> /usr/src/sys/crypto/siphash/siphash.c after SVN r253208.
>>
>> I believe the attached patch is correct but please review ..
>>
>> imb
>
> grr .. missing attachment :-(
Thanks, applied your patch in r253214.
--
Andre
> Index: /usr/src/sys/crypto/siphash/siphash.c
> ===================================================================
> --- /usr/src/sys/crypto/siphash/siphash.c (revision 253210)
> +++ /usr/src/sys/crypto/siphash/siphash.c (working copy)
> @@ -119,7 +119,8 @@
> void
> SipHash_Update(SIPHASH_CTX *ctx, const void *src, size_t len)
> {
> - uint64_t m, *p;
> + uint64_t m;
> + const uint64_t *p;
> const uint8_t *s;
> size_t rem;
>
> @@ -144,13 +145,13 @@
>
> /* Optimze for 64bit aligned/unaligned access. */
> if (((uintptr_t)s & 0x7) == 0) {
> - for (p = (uint64_t *)s; len > 0; len--, p++) {
> + for (p = (const uint64_t *)s; len > 0; len--, p++) {
> m = le64toh(*p);
> ctx->v[3] ^= m;
> SipRounds(ctx, 0);
> ctx->v[0] ^= m;
> }
> - s = (uint8_t *)p;
> + s = (const uint8_t *)p;
> } else {
> for (; len > 0; len--, s += 8) {
> m = le64dec(s);
>
>
> _______________________________________________
> freebsd-current at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe at freebsd.org"
>
>
More information about the freebsd-current
mailing list