RE: [openssl.org #853] serial/index file handling broken on Windows in 0.9.7d
Hi Richard, Sorry for the delay. The snapshot from after you asked me to test it the first time (openssl-0.9.7-stable-SNAP-20040327) passed my tests. Thanks, Steven -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Richard Levitte via RT Sent: Wednesday, 31 March 2004 7:15 AM To: [EMAIL PROTECTED] Cc: [EMAIL PROTECTED] Subject: [openssl.org #853] serial/index file handling broken on Windows in 0.9.7d Hello, I would very much like to know if my fix worked, so I can close this ticket. [levitte - Thu Mar 25 21:10:19 2004]: I think I fixed the problem by moving the definition of Win32_rename() to the end of apps.c. Please check tomorrow's 0.9.7 snapshot and come back to me with any feedback (even hoorays :-)). [EMAIL PROTECTED] - Mon Mar 22 17:17:44 2004]: Re: openssl-dev thread: New CA index subfiles (.attr, .attr.new, . attr.old, .new) OpenSSL 0.9.7d introduces a bug on Windows in the openssl executable in relation to the renaming of temporary serial and index files. The renames are failing since the special WIN32_rename() function (which removes the destination file before the rename) is not being used on Windows due to the rename macro being undefined before it is used. I think the problem was introduced when the serial/index file handling code was tacked onto the end of apps.c. With similar code in a separate module previously the undef was not having such an effect. I've verified that redefining rename by copying line 140 of apps.h and inserting it after line 353 of apps.c fixes all of the problems with serial and index files that I was experiencing: apps.c:352 #undef rename apps.c:353 return rename(from, to); apps.h:140 #define rename(from,to) WIN32_rename((from),(to)) Regards, Steven -- Richard Levitte [EMAIL PROTECTED] __ OpenSSL Project http://www.openssl.org Development Mailing List [EMAIL PROTECTED] Automated List Manager [EMAIL PROTECTED] __ OpenSSL Project http://www.openssl.org Development Mailing List [EMAIL PROTECTED] Automated List Manager [EMAIL PROTECTED]
Re: [openssl.org #853] serial/index file handling broken on Windows in 0.9.7d
In message [EMAIL PROTECTED] on Wed, 31 Mar 2004 12:57:48 +1000, Steven Reddie [EMAIL PROTECTED] said: smr Sorry for the delay. The snapshot from after you asked me to test it the smr first time (openssl-0.9.7-stable-SNAP-20040327) passed my tests. Thanks. - Please consider sponsoring my work on free software. See http://www.free.lp.se/sponsoring.html for details. -- Richard Levitte \ Tunnlandsvägen 52 \ [EMAIL PROTECTED] [EMAIL PROTECTED] \ S-168 36 BROMMA \ T: +46-708-26 53 44 \ SWEDEN \ Procurator Odiosus Ex Infernis-- [EMAIL PROTECTED] Member of the OpenSSL development team: http://www.openssl.org/ Unsolicited commercial email is subject to an archival fee of $400. See http://www.stacken.kth.se/~levitte/mail/ for more info. __ OpenSSL Project http://www.openssl.org Development Mailing List [EMAIL PROTECTED] Automated List Manager [EMAIL PROTECTED]
[openssl.org #853] serial/index file handling broken on Windows in 0.9.7d
Steven Reddie has now confirmed, on openssl-dev, that this issue is resolved. Thanks, Steve. Ticket resolved. [levitte - Tue Mar 30 23:14:37 2004]: I would very much like to know if my fix worked, so I can close this ticket. -- Richard Levitte [EMAIL PROTECTED] __ OpenSSL Project http://www.openssl.org Development Mailing List [EMAIL PROTECTED] Automated List Manager [EMAIL PROTECTED]
[openssl.org #853] serial/index file handling broken on Windows in 0.9.7d
Re: openssl-dev thread: New CA index subfiles (.attr, .attr.new, .attr.old, .new) OpenSSL 0.9.7d introduces a bug on Windows in the openssl executable in relation to the renaming of temporary serial and index files. The renames are failing since the special WIN32_rename() function (which removes the destination file before the rename) is not being used on Windows due to the rename macro being undefined before it is used. I think the problem was introduced when the serial/index file handling code was tacked onto the end of apps.c. With similar code in a separate module previously the undef was not having such an effect. I've verified that redefining rename by copying line 140 of apps.h and inserting it after line 353 of apps.c fixes all of the problems with serial and index files that I was experiencing: apps.c:352 #undef rename apps.c:353 return rename(from, to); apps.h:140 #define rename(from,to) WIN32_rename((from),(to)) Regards, Steven __ OpenSSL Project http://www.openssl.org Development Mailing List [EMAIL PROTECTED] Automated List Manager [EMAIL PROTECTED]