clangTag
ActivePublic

Members

  • This project does not have any members.

Watchers

  • This project does not have any watchers.

Recent Activity

Sep 21 2017

werner closed T3247: SIGILL in prepare_macpads() at md.c:681 as Resolved.

Closing due to compiler error.

Sep 21 2017, 3:39 PM · clang, Ubuntu, libgcrypt, Bug Report

Aug 1 2017

chrullrich added a comment to T3293: libgcrypt: warning: unknown pragma "#pragma GCC optimize" ignored => compile failure with LLVM 5.0.

No, it's not. It still misses "-O" entirely.

Aug 1 2017, 2:26 PM · clang, libgcrypt, Bug Report
cpm closed T3293: libgcrypt: warning: unknown pragma "#pragma GCC optimize" ignored => compile failure with LLVM 5.0 as Resolved.

It's solved!

Aug 1 2017, 2:24 PM · clang, libgcrypt, Bug Report

Jul 24 2017

chrullrich added a comment to T3293: libgcrypt: warning: unknown pragma "#pragma GCC optimize" ignored => compile failure with LLVM 5.0.

The fixed sed expression still does not work correctly; it misses the plain "-O" form of the option. As per gcc docs, -O is the same as -O1, and clang accepts it (and the build falls over with it) even though it does not document it at all.

Jul 24 2017, 10:41 AM · clang, libgcrypt, Bug Report
werner added a commit to T3293: libgcrypt: warning: unknown pragma "#pragma GCC optimize" ignored => compile failure with LLVM 5.0: rCac39522ab08f: random: Fix the command line munging for jitterbase..
Jul 24 2017, 10:01 AM · clang, libgcrypt, Bug Report
werner triaged T3293: libgcrypt: warning: unknown pragma "#pragma GCC optimize" ignored => compile failure with LLVM 5.0 as Low priority.

The warning is just a warning, so no problem. The pragma even indicates the compiler for which it is intended.

Jul 24 2017, 9:41 AM · clang, libgcrypt, Bug Report

Jul 6 2017

jukivili added a comment to T3247: SIGILL in prepare_macpads() at md.c:681.

I did some experimenting and clang SIGILL does not trigger with commonly used, but non-conforming, variable-length object with "struct hack", as below:

Jul 6 2017, 1:22 PM · clang, Ubuntu, libgcrypt, Bug Report

Jul 5 2017

werner added a comment to T3247: SIGILL in prepare_macpads() at md.c:681.

With an integer overflow.

Jul 5 2017, 3:23 PM · clang, Ubuntu, libgcrypt, Bug Report
marcus added a comment to T3247: SIGILL in prepare_macpads() at md.c:681.

This is a standard dynamic sized array:

Jul 5 2017, 3:05 PM · clang, Ubuntu, libgcrypt, Bug Report
werner added a comment to T3247: SIGILL in prepare_macpads() at md.c:681.

Sorry, this is a standard C feature and the only way to have dynamic sized arrays. CLANG simply does not get this pattern right. Grep for pgut001's very comments on such ill behaving compilers (including gcc).

Jul 5 2017, 2:59 PM · clang, Ubuntu, libgcrypt, Bug Report
marcus added a comment to T3247: SIGILL in prepare_macpads() at md.c:681.

At a meta level, I really think that writing more conservative code that enables compilers to do a better job checking for safety is a good idea. The tricks we do with structs are premature optimization from a time when compilers were dumb as a doornail.

Jul 5 2017, 2:56 PM · clang, Ubuntu, libgcrypt, Bug Report
marcus added a comment to T3247: SIGILL in prepare_macpads() at md.c:681.

Maybe casting to a void* helps to disable the check in the compiler.

Jul 5 2017, 2:52 PM · clang, Ubuntu, libgcrypt, Bug Report
gniibe added a comment to T3247: SIGILL in prepare_macpads() at md.c:681.

I can replicate the issue on my system.
It is not the line 681, actually.

Jul 5 2017, 4:07 AM · clang, Ubuntu, libgcrypt, Bug Report

Jul 4 2017

gniibe claimed T3247: SIGILL in prepare_macpads() at md.c:681.

I think that the problem is in your usage with your tool. Please have a look at md_open function in cipher/md.c.
This bug is not the one in libgcrypt, but in the compiler.

Jul 4 2017, 6:27 AM · clang, Ubuntu, libgcrypt, Bug Report
gniibe closed T3246: md5.c:119:3: runtime error: unsigned integer overflow: 2612846078 + 3614090360 cannot be represented in type 'unsigned int' as Invalid.

Same argument can apply to MD5. See T3249: sha256.c:265:3: runtime error: unsigned integer overflow: 4084723048 + 1633837952 cannot be represented in type 'unsigned int' of SHA2.

Jul 4 2017, 5:10 AM · clang, Ubuntu, libgcrypt, Bug Report
gniibe closed T3250: random-csprng.c:1070:16: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'size_t' (aka 'unsigned long') as Invalid.

See T3245: cipher-gcm-intel-pclmul.c:418:17: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'size_t' (aka 'unsigned long').

Jul 4 2017, 5:08 AM · clang, Ubuntu, libgcrypt, Bug Report
gniibe closed T3245: cipher-gcm-intel-pclmul.c:418:17: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'size_t' (aka 'unsigned long') as Invalid.

See T3248: mpiutil.c:501:37: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'unsigned long' for unsigned integer overflow.

Jul 4 2017, 5:06 AM · clang, Ubuntu, libgcrypt, Bug Report
gniibe closed T3248: mpiutil.c:501:37: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'unsigned long' as Invalid.

It is intentionally used.
And in the C programming language, it is defined that unsigned integer never overflows (it is computed as modulo 2).

Jul 4 2017, 4:58 AM · clang, Ubuntu, libgcrypt, Bug Report
gniibe closed T3251: sha512.c:342:25: runtime error: unsigned integer overflow: 17296268247117069635 + 2271443192277056827 cannot be represented in type 'unsigned long' as Invalid.

See T3249: sha256.c:265:3: runtime error: unsigned integer overflow: 4084723048 + 1633837952 cannot be represented in type 'unsigned int'.

Jul 4 2017, 4:54 AM · clang, Ubuntu, libgcrypt, Bug Report
geeknik created T3251: sha512.c:342:25: runtime error: unsigned integer overflow: 17296268247117069635 + 2271443192277056827 cannot be represented in type 'unsigned long'.
Jul 4 2017, 3:11 AM · clang, Ubuntu, libgcrypt, Bug Report
geeknik created T3250: random-csprng.c:1070:16: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'size_t' (aka 'unsigned long').
Jul 4 2017, 3:11 AM · clang, Ubuntu, libgcrypt, Bug Report
geeknik created T3248: mpiutil.c:501:37: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'unsigned long'.
Jul 4 2017, 3:11 AM · clang, Ubuntu, libgcrypt, Bug Report
geeknik created T3247: SIGILL in prepare_macpads() at md.c:681.
Jul 4 2017, 3:11 AM · clang, Ubuntu, libgcrypt, Bug Report
geeknik created T3246: md5.c:119:3: runtime error: unsigned integer overflow: 2612846078 + 3614090360 cannot be represented in type 'unsigned int'.
Jul 4 2017, 3:11 AM · clang, Ubuntu, libgcrypt, Bug Report
geeknik created T3245: cipher-gcm-intel-pclmul.c:418:17: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'size_t' (aka 'unsigned long').
Jul 4 2017, 3:10 AM · clang, Ubuntu, libgcrypt, Bug Report

Mar 30 2017

admin created clang.
Mar 30 2017, 6:42 PM

Nov 8 2012

werner closed T1435: duplicate syms in libmpi as Resolved.
Nov 8 2012, 3:48 PM · clang, gnupg, Bug Report
werner removed a project from T1435: duplicate syms in libmpi: Won't Fix.
Nov 8 2012, 3:47 PM · clang, gnupg, Bug Report
werner added a comment to T1435: duplicate syms in libmpi.

Fixed in git for gnupg 1.4.13, Libgcrypt 1.5.1 and Libgcrypt 1.6.0.

The reason why I was not able to replicate this bug was that
I didn't use -std=c99 with gcc >= 4.3.

Nov 8 2012, 3:47 PM · clang, gnupg, Bug Report

Aug 9 2012

werner added a project to T1432: rijndael implementation contains ambiguos instructions: clang.
Aug 9 2012, 4:04 PM · clang, Bug Report, libgcrypt, Not A Bug
werner added a project to T1431: libgcrypt depends on gnu extensions which hinders clang compilation: clang.
Aug 9 2012, 4:04 PM · clang, Bug Report, libgcrypt
werner closed T1431: libgcrypt depends on gnu extensions which hinders clang compilation as Resolved.
Aug 9 2012, 4:04 PM · clang, Bug Report, libgcrypt
werner added a comment to T1435: duplicate syms in libmpi.

See my comments for T1406. It is clearly a clang bug.

Aug 9 2012, 3:55 PM · clang, gnupg, Bug Report
werner added projects to T1435: duplicate syms in libmpi: Won't Fix, clang.
Aug 9 2012, 3:55 PM · clang, gnupg, Bug Report
werner removed a project from T1406: libmpi inlining results in multiple definitions of symbols (when compiled by clang): patch.
Aug 9 2012, 3:54 PM · Bug Report, libgcrypt
werner added a project to T1406: libmpi inlining results in multiple definitions of symbols (when compiled by clang): clang.
Aug 9 2012, 3:54 PM · Bug Report, libgcrypt