[openssl-commits] [openssl] master update

[Andy Polyakov]

The branch master has been updated
   via  e8c42b9888f53ac60f92221da309dc5a2b007cc3 (commit)
  from  77376c0507b6159b94fc092d7fcd7e7c7a3ed7d9 (commit)


- Log -
commit e8c42b9888f53ac60f92221da309dc5a2b007cc3
Author: Andy Polyakov 
Date:   Fri Feb 23 11:51:41 2018 +0100

ec/asm/x25519-x86_64.pl: fix up ADCX/ADOX fallback.

Reviewed-by: Rich Salz 
Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/5447)

---

Summary of changes:
 crypto/ec/asm/x25519-x86_64.pl | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/crypto/ec/asm/x25519-x86_64.pl b/crypto/ec/asm/x25519-x86_64.pl
index 6c0f738..68a7771 100755
--- a/crypto/ec/asm/x25519-x86_64.pl
+++ b/crypto/ec/asm/x25519-x86_64.pl
@@ -784,7 +784,7 @@ ___
 } else {
 $code.=<<___;
 .globl x25519_fe64_eligible
-.type  x25519_fe64_eligible,\@function
+.type  x25519_fe64_eligible,\@abi-omnipotent
 .align 32
 x25519_fe64_eligible:
xor %eax,%eax
@@ -792,6 +792,7 @@ x25519_fe64_eligible:
 .size  x25519_fe64_eligible,.-x25519_fe64_eligible
 
 .globl x25519_fe64_mul
+.type  x25519_fe64_mul,\@abi-omnipotent
 .globl x25519_fe64_sqr
 .globl x25519_fe64_mul121666
 .globl x25519_fe64_add
@@ -802,9 +803,10 @@ x25519_fe64_sqr:
 x25519_fe64_mul121666:
 x25519_fe64_add:
 x25519_fe64_sub:
-x25519_fe64_sub:
 x25519_fe64_tobytes:
.byte   0x0f,0x0b   # ud2
+   ret
+.size  x25519_fe64_mul,.-x25519_fe64_mul
 ___
 }
 $code.=<<___;
_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits

[openssl-commits] [openssl] master update

[Andy Polyakov]

The branch master has been updated
   via  6afed267db47a8aa604a3a9e78ac72efa02363df (commit)
   via  11a9eacde99f6333707b2399054d66ebbc0eb2a9 (commit)
   via  ae1ffe0f65c460ccdfe5153b96fe9943d7a171b8 (commit)
  from  e8c42b9888f53ac60f92221da309dc5a2b007cc3 (commit)


- Log -
commit 6afed267db47a8aa604a3a9e78ac72efa02363df
Author: Andy Polyakov 
Date:   Fri Feb 23 13:55:37 2018 +0100

ec/ecp_nistp{224,256,521}.c: harmonize usage of __uint128_t.

Reviewed-by: Matt Caswell 
Reviewed-by: Rich Salz 
Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/5449)

commit 11a9eacde99f6333707b2399054d66ebbc0eb2a9
Author: Andy Polyakov 
Date:   Fri Feb 23 13:37:06 2018 +0100

{ec/curve25519,poly1305/poly1305}.c: relax pedantic constraint.

As it turns out gcc -pedantic doesn't seem to consider __uint128_t
as non-standard, unlike __int128 that is.

Fix even MSVC warnings in curve25519.c.

Reviewed-by: Matt Caswell 
Reviewed-by: Rich Salz 
Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/5449)

commit ae1ffe0f65c460ccdfe5153b96fe9943d7a171b8
Author: Andy Polyakov 
Date:   Fri Feb 23 13:20:33 2018 +0100

ec/curve448: portability fixups.

SPARC condition in __SIZEOF_INT128__==16 is rather performance thing
than portability. Even though compiler advertises int128 capability,
corresponding operations are inefficient, because they are not
directly backed by instruction set.

Reviewed-by: Matt Caswell 
Reviewed-by: Rich Salz 
Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/5449)

---

Summary of changes:
 crypto/ec/curve25519.c | 135 ++---
 crypto/ec/curve448/curve448.c  |   2 +-
 crypto/ec/curve448/curve448utils.h |   3 +-
 crypto/ec/ecp_nistp224.c   |   4 +-
 crypto/ec/ecp_nistp256.c   |   4 +-
 crypto/ec/ecp_nistp521.c   |   4 +-
 crypto/poly1305/poly1305.c |   5 +-
 7 files changed, 78 insertions(+), 79 deletions(-)

diff --git a/crypto/ec/curve25519.c b/crypto/ec/curve25519.c
index f354107..45525f4 100644
--- a/crypto/ec/curve25519.c
+++ b/crypto/ec/curve25519.c
@@ -12,9 +12,8 @@
 #include 
 
 #if defined(X25519_ASM) \
-|| ( !defined(PEDANTIC) && \
- !defined(__sparc__) && \
- (defined(__SIZEOF_INT128__) && __SIZEOF_INT128__==16) )
+|| ( (defined(__SIZEOF_INT128__) && __SIZEOF_INT128__ == 16) \
+ && !defined(__sparc__) )
 /*
  * Base 2^51 implementation.
  */
@@ -22,7 +21,7 @@
 
 typedef uint64_t fe51[5];
 # if !defined(X25519_ASM)
-typedef unsigned __int128 u128;
+typedef __uint128_t u128;
 # endif
 
 static const uint64_t MASK51 = 0x7;
@@ -101,38 +100,38 @@ static void fe51_tobytes(uint8_t *s, const fe51 h)
 h4 &= MASK51;
 
 /* smash */
-s[0] = h0 >> 0;
-s[1] = h0 >> 8;
-s[2] = h0 >> 16;
-s[3] = h0 >> 24;
-s[4] = h0 >> 32;
-s[5] = h0 >> 40;
-s[6] = (h0 >> 48) | ((uint32_t)h1 << 3);
-s[7] = h1 >> 5;
-s[8] = h1 >> 13;
-s[9] = h1 >> 21;
-s[10] = h1 >> 29;
-s[11] = h1 >> 37;
-s[12] = (h1 >> 45) | ((uint32_t)h2 << 6);
-s[13] = h2 >> 2;
-s[14] = h2 >> 10;
-s[15] = h2 >> 18;
-s[16] = h2 >> 26;
-s[17] = h2 >> 34;
-s[18] = h2 >> 42;
-s[19] = (h2 >> 50) | ((uint32_t)h3 << 1);
-s[20] = h3 >> 7;
-s[21] = h3 >> 15;
-s[22] = h3 >> 23;
-s[23] = h3 >> 31;
-s[24] = h3 >> 39;
-s[25] = (h3 >> 47) | ((uint32_t)h4 << 4);
-s[26] = h4 >> 4;
-s[27] = h4 >> 12;
-s[28] = h4 >> 20;
-s[29] = h4 >> 28;
-s[30] = h4 >> 36;
-s[31] = h4 >> 44;
+s[0] = (uint8_t)(h0 >> 0);
+s[1] = (uint8_t)(h0 >> 8);
+s[2] = (uint8_t)(h0 >> 16);
+s[3] = (uint8_t)(h0 >> 24);
+s[4] = (uint8_t)(h0 >> 32);
+s[5] = (uint8_t)(h0 >> 40);
+s[6] = (uint8_t)((h0 >> 48) | ((uint32_t)h1 << 3));
+s[7] = (uint8_t)(h1 >> 5);
+s[8] = (uint8_t)(h1 >> 13);
+s[9] = (uint8_t)(h1 >> 21);
+s[10] = (uint8_t)(h1 >> 29);
+s[11] = (uint8_t)(h1 >> 37);
+s[12] = (uint8_t)((h1 >> 45) | ((uint32_t)h2 << 6));
+s[13] = (uint8_t)(h2 >> 2);
+s[14] = (uint8_t)(h2 >> 10);
+s[15] = (uint8_t)(h2 >> 18);
+s[16] = (uint8_t)(h2 >> 26);
+s[17] = (uint8_t)(h2 >> 34);
+s[18] = (uint8_t)(h2 >> 42);
+s[19] = (uint8_t)((h2 >> 50) | ((uint32_t)h3 << 1));
+s[20] = (uint8_t)(h3 >> 7);
+s[21] = (uint8_t)(h3 >> 15);
+s[22] = (uint8_t)(h3 >> 23);
+s[23] = (uint8_t)(h3 >> 31);
+  

[openssl-commits] Errored: openssl/openssl#16585 (master - 441bcaf)

[Travis CI]

Build Update for openssl/openssl
-

Build: #16585
Status: Errored

Duration: 45 minutes and 8 seconds
Commit: 441bcaf (master)
Author: Andy Polyakov
Message: appveyor.yml: omit makedepend step.

makedepend makes lesser sense in a throw-away build like CI, but
it spares some computational time, because with MSVC it takes
separate per-file compiler invocation.

Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/5452)

View the changeset: 
https://github.com/openssl/openssl/compare/6afed267db47...441bcafd8659

View the full build log and details: 
https://travis-ci.org/openssl/openssl/builds/345639117?utm_source=email_medium=notification

--

You can configure recipients for build notifications in your .travis.yml file. 
See https://docs.travis-ci.com/user/notifications

_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits

[openssl-commits] [openssl] master update

[Andy Polyakov]

The branch master has been updated
   via  441bcafd865947474741a71fea49fc5d078b6532 (commit)
  from  6afed267db47a8aa604a3a9e78ac72efa02363df (commit)


- Log -
commit 441bcafd865947474741a71fea49fc5d078b6532
Author: Andy Polyakov 
Date:   Fri Feb 23 17:24:41 2018 +0100

appveyor.yml: omit makedepend step.

makedepend makes lesser sense in a throw-away build like CI, but
it spares some computational time, because with MSVC it takes
separate per-file compiler invocation.

Reviewed-by: Richard Levitte 
(Merged from https://github.com/openssl/openssl/pull/5452)

---

Summary of changes:
 appveyor.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/appveyor.yml b/appveyor.yml
index b3165d5..5074a31 100644
--- a/appveyor.yml
+++ b/appveyor.yml
@@ -22,9 +22,9 @@ before_build:
 }
 - ps: >-
 If ($env:Configuration -Match "shared") {
-$env:SHARED=""
+$env:SHARED="no-makedepend"
 } Else {
-$env:SHARED="no-shared"
+$env:SHARED="no-shared no-makedepend"
 }
 - ps: $env:VSCOMNTOOLS=(Get-Content ("env:VS" + "$env:VSVER" + 
"0COMNTOOLS"))
 - call "%VSCOMNTOOLS%\..\..\VC\vcvarsall.bat" %VCVARS_PLATFORM%
_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits

[openssl-commits] [openssl] OpenSSL_1_1_0-stable update

[Andy Polyakov]

The branch OpenSSL_1_1_0-stable has been updated
   via  831dbc358eab0a50c1b435f93d7c16d99ad61472 (commit)
  from  89314d9a22ad0b0ce87e188c486532926aff01ea (commit)


- Log -
commit 831dbc358eab0a50c1b435f93d7c16d99ad61472
Author: Andy Polyakov 
Date:   Tue Feb 20 12:43:35 2018 +0100

test/recipes/80-test_pkcs12.t: handle lack of Win32::API.

So far check for availability of Win32::API served as implicit check
for $^O being MSWin32. Reportedly it's not safe assumption, and check
for MSWin32 has to be explicit.

Reviewed-by: Matt Caswell 
Reviewed-by: Rich Salz 
(Merged from https://github.com/openssl/openssl/pull/5416)

(cherry picked from commit d4c499f562c1ab7ec7773c3987fc4dce7662a805)

---

Summary of changes:
 test/recipes/80-test_pkcs12.t | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/test/recipes/80-test_pkcs12.t b/test/recipes/80-test_pkcs12.t
index 5bcfb69..aa307d2 100644
--- a/test/recipes/80-test_pkcs12.t
+++ b/test/recipes/80-test_pkcs12.t
@@ -41,6 +41,8 @@ if (eval { require Win32::API; 1; }) {
 SetConsoleOutputCP(1253);
 $pass = Encode::encode("cp1253",Encode::decode("utf-8",$pass));
 }
+} elsif ($^O eq "MSWin32") {
+plan skip_all => "Win32::API unavailable";
 } else {
 # Running MinGW tests transparently under Wine apparently requires
 # UTF-8 locale...
_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits

[openssl-commits] Build failed: openssl OpenSSL_1_1_0-stable.15883

[AppVeyor]

Build openssl OpenSSL_1_1_0-stable.15883 failed


Commit 831dbc358e by Andy Polyakov on 2/24/2018 1:19 PM:

test/recipes/80-test_pkcs12.t: handle lack of Win32::API.


Configure your notification preferences

_
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits