1 From 3fbec7d4db59bbd24a9a768182000a1d004c5bc5 Mon Sep 17 00:00:00 2001
2 From: Tiago Vignatti <tiago.vignatti@nokia.com>
3 Date: Wed, 24 Mar 2010 17:27:43 +0200
4 Subject: [PATCH] Revert "Revert "Render: Use built-in SHA1 library""
6 This reverts commit a39377cbcbd3091095efbeab25bec18ae520147e.
11 include/dix-config.h.in
15 Ok ok ok, it's the revert of a revert! Buhhh...
17 Once upon a time, back in 2007, Carl Worth was trying to boost render
18 (4c6abe1c). He prefered to use a "strong hash" to compare glyphs (19b3b1fd)
19 and used openssl library for this. Further, for the same purpose, people
20 started to set other SHA1 implementations in autoconf. And a lot of
21 alternatives appeared - six, to be precise. In the mean time, John Tapsell
22 commit a builtin implementation of SHA1. In the same day, Keith Packard
23 reverted, stating "X.org should not be providing a custom SHA1
24 implementation." (a39377cb). Now, users ended up with Xorg setting the default
25 as the openssl's one (libcrypto), which takes 88 kB of xserver's private RSS.
26 Besides that, we have a ridiculous "configure dot fucking ac stanza to work
27 out which lib to use is almost as long as sha1.c was", to quote daniels.
29 My simple argument against Keith's decision is simple: we can save 316 kB of
30 RSS in a standalone Xorg call. Therefore, I'm in favor to keep our own very
31 simple and shiny SHA1 implementation.
34 I'm not comfortable yet to send this patch around without see if there's any
35 regressions on render or eventually get some ack from Carl.
38 configure.ac | 89 +------------------------
39 include/dix-config.h.in | 15 ----
41 os/xsha1.c | 168 ---------------------------------------------
42 render/Makefile.am | 3 +
43 render/glyph.c | 25 ++-----
44 render/sha1.c | 173 +++++++++++++++++++++++++++++++++++++++++++++++
45 render/sha1.h | 63 +++++++++++++++++
46 8 files changed, 248 insertions(+), 289 deletions(-)
47 delete mode 100644 os/xsha1.c
48 create mode 100644 render/sha1.c
49 create mode 100644 render/sha1.h
51 diff --git a/configure.ac b/configure.ac
52 index 3e8ea10..ef21aa0 100644
55 @@ -1305,7 +1305,7 @@ AC_DEFINE(BIGREQS, 1, [Support BigRequests extension])
57 if test "x$SPECIAL_DTRACE_OBJECTS" = "xyes" ; then
58 DIX_LIB='$(top_builddir)/dix/dix.O'
59 - OS_LIB='$(top_builddir)/os/os.O $(SHA1_LIBS)'
60 + OS_LIB='$(top_builddir)/os/os.O'
62 DIX_LIB='$(top_builddir)/dix/libdix.la'
63 OS_LIB='$(top_builddir)/os/libos.la'
64 @@ -1325,93 +1325,6 @@ MIEXT_SHADOW_INC='-I$(top_srcdir)/miext/shadow'
65 MIEXT_SHADOW_LIB='$(top_builddir)/miext/shadow/libshadow.la'
66 CORE_INCS='-I$(top_srcdir)/include -I$(top_builddir)/include'
70 - [AS_HELP_STRING([--with-sha1=libc|libmd|libgcrypt|libcrypto|libsha1|CommonCrypto],
71 - [choose SHA1 implementation])])
72 -AC_CHECK_FUNC([SHA1Init], [HAVE_SHA1_IN_LIBC=yes])
73 -if test "x$with_sha1" = x && test "x$HAVE_SHA1_IN_LIBC" = xyes; then
76 -if test "x$with_sha1" = xlibc && test "x$HAVE_SHA1_IN_LIBC" != xyes; then
77 - AC_MSG_ERROR([libc requested but not found])
79 -if test "x$with_sha1" = xlibc; then
80 - AC_DEFINE([HAVE_SHA1_IN_LIBC], [1],
81 - [Use libc SHA1 functions])
84 -AC_CHECK_FUNC([CC_SHA1_Init], [HAVE_SHA1_IN_COMMONCRYPTO=yes])
85 -if test "x$with_sha1" = x && test "x$HAVE_SHA1_IN_COMMONCRYPTO" = xyes; then
86 - with_sha1=CommonCrypto
88 -if test "x$with_sha1" = xCommonCrypto && test "x$HAVE_SHA1_IN_COMMONCRYPTO" != xyes; then
89 - AC_MSG_ERROR([CommonCrypto requested but not found])
91 -if test "x$with_sha1" = xCommonCrypto; then
92 - AC_DEFINE([HAVE_SHA1_IN_COMMONCRYPTO], [1],
93 - [Use CommonCrypto SHA1 functions])
96 -AC_CHECK_LIB([md], [SHA1Init], [HAVE_LIBMD=yes])
97 -if test "x$with_sha1" = x && test "x$HAVE_LIBMD" = xyes; then
100 -if test "x$with_sha1" = xlibmd && test "x$HAVE_LIBMD" != xyes; then
101 - AC_MSG_ERROR([libmd requested but not found])
103 -if test "x$with_sha1" = xlibmd; then
104 - AC_DEFINE([HAVE_SHA1_IN_LIBMD], [1],
105 - [Use libmd SHA1 functions])
108 -AC_CHECK_LIB([gcrypt], [gcry_md_open], [HAVE_LIBGCRYPT=yes])
109 -if test "x$with_sha1" = x && test "x$HAVE_LIBGCRYPT" = xyes; then
110 - with_sha1=libgcrypt
112 -if test "x$with_sha1" = xlibgcrypt; then
113 - AC_DEFINE([HAVE_SHA1_IN_LIBGCRYPT], [1],
114 - [Use libgcrypt SHA1 functions])
117 -AC_CHECK_LIB([sha1], [sha1_begin], [HAVE_LIBSHA1=yes])
118 -if test "x$with_sha1" = x && test "x$HAVE_LIBSHA1" = xyes; then
121 -if test "x$with_sha1" = xlibsha1; then
122 - AC_DEFINE([HAVE_SHA1_IN_LIBSHA1], [1],
123 - [Use libsha1 for SHA1])
126 -# We don't need all of the OpenSSL libraries, just libcrypto
127 -AC_CHECK_LIB([crypto], [SHA1_Init], [HAVE_LIBCRYPTO=yes])
128 -PKG_CHECK_MODULES([OPENSSL], [openssl], [HAVE_OPENSSL_PKC=yes],
129 - [HAVE_OPENSSL_PKC=no])
130 -if test "x$HAVE_LIBCRYPTO" = xyes || test "x$HAVE_OPENSSL_PKC" = xyes; then
131 - if test "x$with_sha1" = x; then
132 - with_sha1=libcrypto
135 - if test "x$with_sha1" = xlibcrypto; then
136 - AC_MSG_ERROR([OpenSSL libcrypto requested but not found])
139 -if test "x$with_sha1" = xlibcrypto; then
140 - if test "x$HAVE_LIBCRYPTO" = xyes; then
143 - SHA1_LIBS="$OPENSSL_LIBS"
144 - SHA1_CFLAGS="$OPENSSL_CFLAGS"
147 -AC_MSG_CHECKING([for SHA1 implementation])
148 -if test "x$with_sha1" = x; then
149 - AC_MSG_ERROR([No suitable SHA1 implementation found])
151 -AC_MSG_RESULT([$with_sha1])
153 -AC_SUBST(SHA1_CFLAGS)
155 PKG_CHECK_MODULES([XSERVERCFLAGS], [$REQUIRED_MODULES $REQUIRED_LIBS])
156 PKG_CHECK_MODULES([XSERVERLIBS], [$REQUIRED_LIBS])
158 diff --git a/include/dix-config.h.in b/include/dix-config.h.in
159 index 058c8fd..d6e99a5 100644
160 --- a/include/dix-config.h.in
161 +++ b/include/dix-config.h.in
163 /* Define to 1 if you have the <rpcsvc/dbm.h> header file. */
164 #undef HAVE_RPCSVC_DBM_H
166 -/* Define to use libc SHA1 functions */
167 -#undef HAVE_SHA1_IN_LIBC
169 -/* Define to use CommonCrypto SHA1 functions */
170 -#undef HAVE_SHA1_IN_COMMONCRYPTO
172 -/* Define to use libmd SHA1 functions */
173 -#undef HAVE_SHA1_IN_LIBMD
175 -/* Define to use libgcrypt SHA1 functions */
176 -#undef HAVE_SHA1_IN_LIBGCRYPT
178 -/* Define to use libsha1 for SHA1 */
179 -#undef HAVE_SHA1_IN_LIBSHA1
181 /* Define to 1 if you have the `shmctl64' function. */
184 diff --git a/os/Makefile.am b/os/Makefile.am
185 index 66a4a0f..b8c1636 100644
188 @@ -22,7 +22,6 @@ libos_la_SOURCES = \
196 diff --git a/os/xsha1.c b/os/xsha1.c
197 deleted file mode 100644
198 index 355862f..0000000
202 -#ifdef HAVE_DIX_CONFIG_H
203 -#include <dix-config.h>
209 -#if defined(HAVE_SHA1_IN_LIBMD) /* Use libmd for SHA1 */ \
210 - || defined(HAVE_SHA1_IN_LIBC) /* Use libc for SHA1 */
214 -void *x_sha1_init(void)
216 - SHA1_CTX *ctx = xalloc(sizeof(*ctx));
223 -int x_sha1_update(void *ctx, void *data, int size)
225 - SHA1_CTX *sha1_ctx = ctx;
226 - SHA1Update(sha1_ctx, data, size);
230 -int x_sha1_final(void *ctx, unsigned char result[20])
232 - SHA1_CTX *sha1_ctx = ctx;
233 - SHA1Final(result, sha1_ctx);
238 -#elif defined(HAVE_SHA1_IN_COMMONCRYPTO) /* Use CommonCrypto for SHA1 */
240 -#include <CommonCrypto/CommonDigest.h>
242 -void *x_sha1_init(void)
244 - CC_SHA1_CTX *ctx = xalloc(sizeof(*ctx));
251 -int x_sha1_update(void *ctx, void *data, int size)
253 - CC_SHA1_CTX *sha1_ctx = ctx;
254 - CC_SHA1_Update(sha1_ctx, data, size);
258 -int x_sha1_final(void *ctx, unsigned char result[20])
260 - CC_SHA1_CTX *sha1_ctx = ctx;
261 - CC_SHA1_Final(result, sha1_ctx);
266 -#elif defined(HAVE_SHA1_IN_LIBGCRYPT) /* Use libgcrypt for SHA1 */
268 -# include <gcrypt.h>
270 -void *x_sha1_init(void)
277 - if (!gcry_check_version(NULL))
279 - gcry_control(GCRYCTL_DISABLE_SECMEM, 0);
280 - gcry_control(GCRYCTL_INITIALIZATION_FINISHED, 0);
284 - err = gcry_md_open(&h, GCRY_MD_SHA1, 0);
290 -int x_sha1_update(void *ctx, void *data, int size)
292 - gcry_md_hd_t h = ctx;
293 - gcry_md_write(h, data, size);
297 -int x_sha1_final(void *ctx, unsigned char result[20])
299 - gcry_md_hd_t h = ctx;
300 - memcpy(result, gcry_md_read(h, GCRY_MD_SHA1), 20);
305 -#elif defined(HAVE_SHA1_IN_LIBSHA1) /* Use libsha1 */
307 -# include <libsha1.h>
309 -void *x_sha1_init(void)
311 - sha1_ctx *ctx = xalloc(sizeof(*ctx));
318 -int x_sha1_update(void *ctx, void *data, int size)
320 - sha1_hash(data, size, ctx);
324 -int x_sha1_final(void *ctx, unsigned char result[20])
326 - sha1_end(result, ctx);
331 -#else /* Use OpenSSL's libcrypto */
333 -# include <stddef.h> /* buggy openssl/sha.h wants size_t */
334 -# include <openssl/sha.h>
336 -void *x_sha1_init(void)
339 - SHA_CTX *ctx = xalloc(sizeof(*ctx));
342 - ret = SHA1_Init(ctx);
350 -int x_sha1_update(void *ctx, void *data, int size)
353 - SHA_CTX *sha_ctx = ctx;
354 - ret = SHA1_Update(sha_ctx, data, size);
360 -int x_sha1_final(void *ctx, unsigned char result[20])
363 - SHA_CTX *sha_ctx = ctx;
364 - ret = SHA1_Final(result, sha_ctx);
370 diff --git a/render/Makefile.am b/render/Makefile.am
371 index 216c613..bb46702 100644
372 --- a/render/Makefile.am
373 +++ b/render/Makefile.am
374 @@ -14,8 +14,11 @@ librender_la_SOURCES = \
382 sdk_HEADERS = picture.h mipict.h glyphstr.h picturestr.h renderedge.h
386 diff --git a/render/glyph.c b/render/glyph.c
387 index 0b864ad..e14530a 100644
391 #include <dix-config.h>
398 #include "scrnintstr.h"
400 @@ -193,21 +192,13 @@ HashGlyph (xGlyphInfo *gi,
402 unsigned char sha1[20])
404 - void *ctx = x_sha1_init();
410 - success = x_sha1_update(ctx, gi, sizeof(xGlyphInfo));
413 - success = x_sha1_update(ctx, bits, size);
416 - success = x_sha1_final(ctx, sha1);
422 + SHA1Update (&ctx, gi, sizeof (xGlyphInfo));
423 + SHA1Update (&ctx, bits, size);
424 + SHA1Final (sha1, &ctx);
429 diff --git a/render/sha1.c b/render/sha1.c
431 index 0000000..820eb2a
437 + * By Steve Reid <steve@edmweb.com>
438 + * 100% Public Domain
440 + * Test Vectors (from FIPS PUB 180-1)
442 + * A9993E36 4706816A BA3E2571 7850C26C 9CD0D89D
443 + * "abcdbcdecdefdefgefghfghighijhijkijkljklmklmnlmnomnopnopq"
444 + * 84983E44 1C3BD26E BAAE4AA1 F95129E5 E54670F1
445 + * A million repetitions of "a"
446 + * 34AA973C D4C4DAA4 F61EEB2B DBAD2731 6534016F
449 +#include <sys/param.h>
453 +#define rol(value, bits) (((value) << (bits)) | ((value) >> (32 - (bits))))
456 + * blk0() and blk() perform the initial expand.
457 + * I got the idea of expanding during the round function from SSLeay
459 +#if BYTE_ORDER == LITTLE_ENDIAN
460 +# define blk0(i) (block->l[i] = (rol(block->l[i],24)&0xFF00FF00) \
461 + |(rol(block->l[i],8)&0x00FF00FF))
463 +# define blk0(i) block->l[i]
465 +#define blk(i) (block->l[i&15] = rol(block->l[(i+13)&15]^block->l[(i+8)&15] \
466 + ^block->l[(i+2)&15]^block->l[i&15],1))
469 + * (R0+R1), R2, R3, R4 are the different operations (rounds) used in SHA1
471 +#define R0(v,w,x,y,z,i) z+=((w&(x^y))^y)+blk0(i)+0x5A827999+rol(v,5);w=rol(w,30);
472 +#define R1(v,w,x,y,z,i) z+=((w&(x^y))^y)+blk(i)+0x5A827999+rol(v,5);w=rol(w,30);
473 +#define R2(v,w,x,y,z,i) z+=(w^x^y)+blk(i)+0x6ED9EBA1+rol(v,5);w=rol(w,30);
474 +#define R3(v,w,x,y,z,i) z+=(((w|x)&y)|(w&x))+blk(i)+0x8F1BBCDC+rol(v,5);w=rol(w,30);
475 +#define R4(v,w,x,y,z,i) z+=(w^x^y)+blk(i)+0xCA62C1D6+rol(v,5);w=rol(w,30);
478 + * Hash a single 512-bit block. This is the core of the algorithm.
481 +SHA1Transform(uint32_t state[5], const uint8_t buffer[SHA1_BLOCK_LENGTH])
483 + uint32_t a, b, c, d, e;
484 + uint8_t workspace[SHA1_BLOCK_LENGTH];
489 + CHAR64LONG16 *block = (CHAR64LONG16 *)workspace;
491 + (void)memcpy(block, buffer, SHA1_BLOCK_LENGTH);
493 + /* Copy context->state[] to working vars */
500 + /* 4 rounds of 20 operations each. Loop unrolled. */
501 + R0(a,b,c,d,e, 0); R0(e,a,b,c,d, 1); R0(d,e,a,b,c, 2); R0(c,d,e,a,b, 3);
502 + R0(b,c,d,e,a, 4); R0(a,b,c,d,e, 5); R0(e,a,b,c,d, 6); R0(d,e,a,b,c, 7);
503 + R0(c,d,e,a,b, 8); R0(b,c,d,e,a, 9); R0(a,b,c,d,e,10); R0(e,a,b,c,d,11);
504 + R0(d,e,a,b,c,12); R0(c,d,e,a,b,13); R0(b,c,d,e,a,14); R0(a,b,c,d,e,15);
505 + R1(e,a,b,c,d,16); R1(d,e,a,b,c,17); R1(c,d,e,a,b,18); R1(b,c,d,e,a,19);
506 + R2(a,b,c,d,e,20); R2(e,a,b,c,d,21); R2(d,e,a,b,c,22); R2(c,d,e,a,b,23);
507 + R2(b,c,d,e,a,24); R2(a,b,c,d,e,25); R2(e,a,b,c,d,26); R2(d,e,a,b,c,27);
508 + R2(c,d,e,a,b,28); R2(b,c,d,e,a,29); R2(a,b,c,d,e,30); R2(e,a,b,c,d,31);
509 + R2(d,e,a,b,c,32); R2(c,d,e,a,b,33); R2(b,c,d,e,a,34); R2(a,b,c,d,e,35);
510 + R2(e,a,b,c,d,36); R2(d,e,a,b,c,37); R2(c,d,e,a,b,38); R2(b,c,d,e,a,39);
511 + R3(a,b,c,d,e,40); R3(e,a,b,c,d,41); R3(d,e,a,b,c,42); R3(c,d,e,a,b,43);
512 + R3(b,c,d,e,a,44); R3(a,b,c,d,e,45); R3(e,a,b,c,d,46); R3(d,e,a,b,c,47);
513 + R3(c,d,e,a,b,48); R3(b,c,d,e,a,49); R3(a,b,c,d,e,50); R3(e,a,b,c,d,51);
514 + R3(d,e,a,b,c,52); R3(c,d,e,a,b,53); R3(b,c,d,e,a,54); R3(a,b,c,d,e,55);
515 + R3(e,a,b,c,d,56); R3(d,e,a,b,c,57); R3(c,d,e,a,b,58); R3(b,c,d,e,a,59);
516 + R4(a,b,c,d,e,60); R4(e,a,b,c,d,61); R4(d,e,a,b,c,62); R4(c,d,e,a,b,63);
517 + R4(b,c,d,e,a,64); R4(a,b,c,d,e,65); R4(e,a,b,c,d,66); R4(d,e,a,b,c,67);
518 + R4(c,d,e,a,b,68); R4(b,c,d,e,a,69); R4(a,b,c,d,e,70); R4(e,a,b,c,d,71);
519 + R4(d,e,a,b,c,72); R4(c,d,e,a,b,73); R4(b,c,d,e,a,74); R4(a,b,c,d,e,75);
520 + R4(e,a,b,c,d,76); R4(d,e,a,b,c,77); R4(c,d,e,a,b,78); R4(b,c,d,e,a,79);
522 + /* Add the working vars back into context.state[] */
529 + /* Wipe variables */
530 + a = b = c = d = e = 0;
535 + * SHA1Init - Initialize new context
538 +SHA1Init(SHA1_CTX *context)
541 + /* SHA1 initialization constants */
542 + context->count = 0;
543 + context->state[0] = 0x67452301;
544 + context->state[1] = 0xEFCDAB89;
545 + context->state[2] = 0x98BADCFE;
546 + context->state[3] = 0x10325476;
547 + context->state[4] = 0xC3D2E1F0;
552 + * Run your data through this.
555 +SHA1Update(SHA1_CTX *context, const uint8_t *data, size_t len)
559 + j = (size_t)((context->count >> 3) & 63);
560 + context->count += (len << 3);
561 + if ((j + len) > 63) {
562 + (void)memcpy(&context->buffer[j], data, (i = 64-j));
563 + SHA1Transform(context->state, context->buffer);
564 + for ( ; i + 63 < len; i += 64)
565 + SHA1Transform(context->state, (uint8_t *)&data[i]);
570 + (void)memcpy(&context->buffer[j], &data[i], len - i);
575 + * Add padding and return the message digest.
578 +SHA1Pad(SHA1_CTX *context)
580 + uint8_t finalcount[8];
583 + for (i = 0; i < 8; i++) {
584 + finalcount[i] = (uint8_t)((context->count >>
585 + ((7 - (i & 7)) * 8)) & 255); /* Endian independent */
587 + SHA1Update(context, (uint8_t *)"\200", 1);
588 + while ((context->count & 504) != 448)
589 + SHA1Update(context, (uint8_t *)"\0", 1);
590 + SHA1Update(context, finalcount, 8); /* Should cause a SHA1Transform() */
594 +SHA1Final(uint8_t digest[SHA1_DIGEST_LENGTH], SHA1_CTX *context)
600 + for (i = 0; i < SHA1_DIGEST_LENGTH; i++) {
601 + digest[i] = (uint8_t)
602 + ((context->state[i>>2] >> ((3-(i & 3)) * 8) ) & 255);
604 + memset(context, 0, sizeof(*context));
608 diff --git a/render/sha1.h b/render/sha1.h
610 index 0000000..ace7d97
616 + * By Steve Reid <steve@edmweb.com>
617 + * 100% Public Domain
628 +#define SHA1_BLOCK_LENGTH 64
629 +#define SHA1_DIGEST_LENGTH 20
630 +#define SHA1_DIGEST_STRING_LENGTH (SHA1_DIGEST_LENGTH * 2 + 1)
635 + uint8_t buffer[SHA1_BLOCK_LENGTH];
638 +#include <sys/cdefs.h>
641 +void SHA1Init(SHA1_CTX *);
642 +void SHA1Pad(SHA1_CTX *);
643 +void SHA1Transform(uint32_t [5], const uint8_t [SHA1_BLOCK_LENGTH])
644 + __attribute__((__bounded__(__minbytes__,1,5)))
645 + __attribute__((__bounded__(__minbytes__,2,SHA1_BLOCK_LENGTH)));
646 +void SHA1Update(SHA1_CTX *, const uint8_t *, size_t)
647 + __attribute__((__bounded__(__string__,2,3)));
648 +void SHA1Final(uint8_t [SHA1_DIGEST_LENGTH], SHA1_CTX *)
649 + __attribute__((__bounded__(__minbytes__,1,SHA1_DIGEST_LENGTH)));
650 +char *SHA1End(SHA1_CTX *, char *)
651 + __attribute__((__bounded__(__minbytes__,2,SHA1_DIGEST_STRING_LENGTH)));
652 +char *SHA1File(const char *, char *)
653 + __attribute__((__bounded__(__minbytes__,2,SHA1_DIGEST_STRING_LENGTH)));
654 +char *SHA1FileChunk(const char *, char *, off_t, off_t)
655 + __attribute__((__bounded__(__minbytes__,2,SHA1_DIGEST_STRING_LENGTH)));
656 +char *SHA1Data(const uint8_t *, size_t, char *)
657 + __attribute__((__bounded__(__string__,1,2)))
658 + __attribute__((__bounded__(__minbytes__,3,SHA1_DIGEST_STRING_LENGTH)));
661 +#define HTONDIGEST(x) do { \
662 + x[0] = htonl(x[0]); \
663 + x[1] = htonl(x[1]); \
664 + x[2] = htonl(x[2]); \
665 + x[3] = htonl(x[3]); \
666 + x[4] = htonl(x[4]); } while (0)
668 +#define NTOHDIGEST(x) do { \
669 + x[0] = ntohl(x[0]); \
670 + x[1] = ntohl(x[1]); \
671 + x[2] = ntohl(x[2]); \
672 + x[3] = ntohl(x[3]); \
673 + x[4] = ntohl(x[4]); } while (0)
675 +#endif /* _SHA1_H */