Mailing List Archive

www.netbsd.se

OpenSSL


List view - Thread view


Subject: [openssl.org #1681] 0.9.8h bug report

From: Craig Muchinsky via RT

Date: 2008-05-29 08:13:00
















I pulled down 0.9.8h today and attempted to build on a win32
machine, but ran into an issue when compiling the generated s1_win32.asm file.
It looks like there is a syntax error in sha1-586.pl at line 152, the second
argument (16) is causing the following error:



 ml /Cp /coff /c /Cx
/Focrypto\sha\asm\s1_win32.obj .\crypto\sha\asm\s1_win32.asm

 Assembling: .\crypto\sha\asm\s1_win32.asm

Microsoft (R) Macro Assembler Version
8.00.50727.762

Copyright (C) Microsoft Corporation. All rights
reserved.

.\crypto\sha\asm\s1_win32.asm(13) : error A2008:
syntax error : integer

 

NMAKE : fatal error U1077: '"C:\Program Files
(x86)\Microsoft Visual Studio 8\VC\bin\ml.EXE"' : return code '0x1'

Stop.



By simply removing the ‘,16’ from line 152
everything compiles fine.



Thanks,

Craig Muchinsky 







From: Mounir IDRASSI

Date: 2008-05-29 10:49:33


Hi,

You should not touch the file sha1-586.pl because the problem is located
in the file x86ms.pl that is dedicated to MASM. In this file, the line 273
containing "$extra" should be removed to be able to compile the generated
assembly files.

Cheers,
-- 
Mounir IDRASSI
IDRIX
http://www.idrix.fr

On Thu, May 29, 2008 8:13 am, Craig Muchinsky via RT wrote:
> I pulled down 0.9.8h today and attempted to build on a win32 machine,
> but ran into an issue when compiling the generated s1_win32.asm file. It
> looks like there is a syntax error in sha1-586.pl at line 152, the
> second argument (16) is causing the following error:
>
>
>
>      ml /Cp /coff /c /Cx /Focrypto\sha\asm\s1_win32.obj
> .\crypto\sha\asm\s1_win32.asm
>
>    Assembling: .\crypto\sha\asm\s1_win32.asm
>
>   Microsoft (R) Macro Assembler Version 8.00.50727.762
>
>   Copyright (C) Microsoft Corporation.  All rights reserved.
>
>   .\crypto\sha\asm\s1_win32.asm(13) : error A2008: syntax error :
> integer
>
>
>
>   NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual
> Studio 8\VC\bin\ml.EXE"' : return code '0x1'
>
>   Stop.
>
>
>
> By simply removing the ',16' from line 152 everything compiles fine.
>
>
>
> Thanks,
>
> Craig Muchinsky
>

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       openssl-<email removed>
Automated List Manager                           <email removed>

From: Mounir IDRASSI via RT

Date: 2008-05-29 09:59:46


Hi,

You should not touch the file sha1-586.pl because the problem is located
in the file x86ms.pl that is dedicated to MASM. In this file, the line 273
containing "$extra" should be removed to be able to compile the generated
assembly files.

Cheers,
-- 
Mounir IDRASSI
IDRIX
http://www.idrix.fr

On Thu, May 29, 2008 8:13 am, Craig Muchinsky via RT wrote:
> I pulled down 0.9.8h today and attempted to build on a win32 machine,
> but ran into an issue when compiling the generated s1_win32.asm file. It
> looks like there is a syntax error in sha1-586.pl at line 152, the
> second argument (16) is causing the following error:
>
>
>
>      ml /Cp /coff /c /Cx /Focrypto\sha\asm\s1_win32.obj
> .\crypto\sha\asm\s1_win32.asm
>
>    Assembling: .\crypto\sha\asm\s1_win32.asm
>
>   Microsoft (R) Macro Assembler Version 8.00.50727.762
>
>   Copyright (C) Microsoft Corporation.  All rights reserved.
>
>   .\crypto\sha\asm\s1_win32.asm(13) : error A2008: syntax error :
> integer
>
>
>
>   NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual
> Studio 8\VC\bin\ml.EXE"' : return code '0x1'
>
>   Stop.
>
>
>
> By simply removing the ',16' from line 152 everything compiles fine.
>
>
>
> Thanks,
>
> Craig Muchinsky
>


______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       openssl-<email removed>
Automated List Manager                           <email removed>

From: Andy Polyakov via RT

Date: 2008-07-17 13:44:49


> You should not touch the file sha1-586.pl

sha1-586.pl was back-ported from in-development HEAD branch and extra
argument is a left-over and *should not* be there.

> because the problem is located
> in the file x86ms.pl that is dedicated to MASM. In this file, the line 273
> containing "$extra" should be removed to be able to compile the \ 
generated
> assembly files.

$extra is used in another modules. They, other modules, *might* work
without it now, but it used to be required. Given *might*/*should not*
relationship, it's extra argument of 16 in sha1-586.pl that should go,
not $extra in x86ms.pl. Changes are committed and case is being
dismissed. A.


______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       openssl-<email removed>
Automated List Manager                           <email removed>