Bug 948074 - [abrt] mingw32-nsis-2.46-8.fc18: CResourceEditor::WriteRsrcSec: Process /usr/bin/makensis was killed by signal 11 (SIGSEGV) [NEEDINFO]
Summary: [abrt] mingw32-nsis-2.46-8.fc18: CResourceEditor::WriteRsrcSec: Process /usr/...
Alias: None
Product: Fedora
Classification: Fedora
Component: mingw-nsis
Version: 19
Hardware: x86_64
OS: Unspecified
Target Milestone: ---
Assignee: Kevin Kofler
QA Contact: Fedora Extras Quality Assurance
Whiteboard: abrt_hash:765aaae6e329e561c04b10e1db9...
: 967932 1026320 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2013-04-03 22:35 UTC by Richard Foster
Modified: 2015-09-28 09:31 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2015-02-17 14:56:21 UTC
urkle: needinfo?

Attachments (Terms of Use)
File: backtrace (90.86 KB, text/plain)
2013-04-03 22:35 UTC, Richard Foster
no flags Details
File: cgroup (130 bytes, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: core_backtrace (697 bytes, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: dso_list (842 bytes, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: environ (214 bytes, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: limits (1.29 KB, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: maps (3.97 KB, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: open_fds (160 bytes, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: proc_pid_status (935 bytes, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: smolt_data (4.39 KB, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
File: var_log_messages (313 bytes, text/plain)
2013-04-03 22:36 UTC, Richard Foster
no flags Details
Patch to fix crash in 64bit build of makensis (574 bytes, patch)
2014-07-30 15:45 UTC, Edward Rudd
no flags Details | Diff

Description Richard Foster 2013-04-03 22:35:54 UTC
Description of problem:
Honestly not sure - believe I was building LibUpnpCil from http://oss.linn.co.uk/repos/Public, I think it crashed trying to compile for windows, which wasn't necessary.

Version-Release number of selected component:

Additional info:
backtrace_rating: 4
cmdline:        makensis -V2 /home/richard/Programming/Linn/build/Linux/release/Control/UpnpDiagnostics.nsi
crash_function: CResourceEditor::WriteRsrcSec
executable:     /usr/bin/makensis
kernel:         3.8.4-202.fc18.x86_64
uid:            1000
ureports_counter: 1

Truncated backtrace:
Thread no. 1 (4 frames)
 #0 CResourceEditor::WriteRsrcSec at Source/ResourceEditor.cpp:757
 #1 CResourceEditor::Save at Source/ResourceEditor.cpp:514
 #2 CEXEBuild::close_res_editor at Source/build.cpp:3379
 #3 CEXEBuild::write_output at Source/build.cpp:2449

Comment 1 Richard Foster 2013-04-03 22:35:57 UTC
Created attachment 731421 [details]
File: backtrace

Comment 2 Richard Foster 2013-04-03 22:36:00 UTC
Created attachment 731422 [details]
File: cgroup

Comment 3 Richard Foster 2013-04-03 22:36:02 UTC
Created attachment 731423 [details]
File: core_backtrace

Comment 4 Richard Foster 2013-04-03 22:36:04 UTC
Created attachment 731424 [details]
File: dso_list

Comment 5 Richard Foster 2013-04-03 22:36:06 UTC
Created attachment 731425 [details]
File: environ

Comment 6 Richard Foster 2013-04-03 22:36:08 UTC
Created attachment 731426 [details]
File: limits

Comment 7 Richard Foster 2013-04-03 22:36:10 UTC
Created attachment 731427 [details]
File: maps

Comment 8 Richard Foster 2013-04-03 22:36:12 UTC
Created attachment 731428 [details]
File: open_fds

Comment 9 Richard Foster 2013-04-03 22:36:13 UTC
Created attachment 731429 [details]
File: proc_pid_status

Comment 10 Richard Foster 2013-04-03 22:36:15 UTC
Created attachment 731430 [details]
File: smolt_data

Comment 11 Richard Foster 2013-04-03 22:36:17 UTC
Created attachment 731431 [details]
File: var_log_messages

Comment 12 Kevin Kofler 2013-04-04 02:19:26 UTC
Hmmm, this might be due to my patches to make native 64-bit builds of the host parts of NSIS possible. I'll have a look ASAP to see what could be wrong.

Comment 13 Kevin Kofler 2013-04-04 03:00:06 UTC
Yeah, at least those m_dwWrittenAt variables need to be changed from DWORD to long too, they're used to store host pointers.

Comment 14 Kevin Kofler 2013-04-04 03:09:04 UTC
This could be of interest:
They target W64, which means they probably have fixes for the stuff I fixed and also for the stuff I missed, and they probably also have 64-bit TARGET support working, which I didn't even attempt, so we should be able build a mingw64-nsis package from that. But I haven't tried it yet, so I don't know whether they haven't broken POSIX host support or some other thing we need.

Comment 15 Kevin Kofler 2013-04-04 03:28:29 UTC
Actually, this particular issue was fixed upstream (in the official NSIS, not the nsis64 fork) as part of:

Comment 16 Kevin Kofler 2013-04-07 13:54:28 UTC
I'm still working on this. I don't want to apply revision 6040 as is because:
* it contains unrelated changes (countof, Unicode/tchar.h) and
* it does not contain this change:
@@ -703,7 +703,7 @@
   WORD e_oemid;
   WORD e_oeminfo;
   WORD e_res2[10];
-  LONG e_lfanew;
+  DWORD e_lfanew;
 #  pragma pack()
 #  pragma pack(4)
which I'm fairly sure is needed. IMAGE_DOS_HEADER is a standard structure which is even getting defined up to packing, having e_lfanew be a different size on different hosts sounds like a very bad idea to me.

BTW, a recent upstream commit:
changes DWORD to be unsigned long, which looks even more broken to me, I doubt that commit was even tested at all on an LP64 platform.

Comment 17 Kevin Kofler 2013-05-29 22:32:30 UTC
*** Bug 967932 has been marked as a duplicate of this bug. ***

Comment 18 Kevin Kofler 2013-11-12 00:08:52 UTC
Whoops, I dropped the ball on this one, I'll have another look ASAP.

Comment 19 Kevin Kofler 2013-11-12 00:09:41 UTC
*** Bug 1026320 has been marked as a duplicate of this bug. ***

Comment 20 Edward Rudd 2014-07-23 21:25:08 UTC
According to the LLVM source DWORD would be correct for e_lfanew..  (reference: http://llvm.org/docs/doxygen/html/Object_2COFF_8h_source.html ).

...time passes...

Actually I wend through and debugged the binary and located the issue and have it fixed!

it's REALLY super simple too..

on line 757 of ResourceEditor.cpp the cRDataE->m_dwWrittenAt member is cast as a pointer of a struct.     However..  m_dwWrittenAt is a DWORD (32bits) thus it fails....  horribly...  resulting in this segfault.

This patch fixes the issue..

--- ResourceEditor.h.64bit	2009-02-01 09:44:30.000000000 -0500
+++ ResourceEditor.h	2014-07-23 17:12:20.000000000 -0400
@@ -180,7 +180,7 @@
   void Destroy();
-  DWORD m_dwWrittenAt;
+  long m_dwWrittenAt;
@@ -204,7 +204,7 @@
   CResourceDataEntry* GetDataEntry();
-  DWORD m_dwWrittenAt;
+  long m_dwWrittenAt;
   bool m_bHasName;
@@ -232,7 +232,7 @@
   DWORD GetCodePage();
   DWORD GetOffset();
-  DWORD m_dwWrittenAt;
+  long m_dwWrittenAt;
   BYTE* m_pbData;

Comment 21 Edward Rudd 2014-07-30 15:45:04 UTC
Created attachment 922639 [details]
Patch to fix crash in 64bit build of makensis

Comment 22 Fedora End Of Life 2015-01-09 17:51:05 UTC
This message is a notice that Fedora 19 is now at end of life. Fedora 
has stopped maintaining and issuing updates for Fedora 19. It is 
Fedora's policy to close all bug reports from releases that are no 
longer maintained. Approximately 4 (four) weeks from now this bug will
be closed as EOL if it remains open with a Fedora 'version' of '19'.

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, simply change the 'version' 
to a later Fedora version.

Thank you for reporting this issue and we are sorry that we were not 
able to fix it before Fedora 19 is end of life. If you would still like 
to see this bug fixed and are able to reproduce it against a later version 
of Fedora, you are encouraged  change the 'version' to a later Fedora 
version prior this bug is closed as described in the policy above.

Although we aim to fix as many bugs as possible during every release's 
lifetime, sometimes those efforts are overtaken by events. Often a 
more recent Fedora release includes newer upstream software that fixes 
bugs or makes them obsolete.

Comment 23 Fedora End Of Life 2015-02-17 14:56:21 UTC
Fedora 19 changed to end-of-life (EOL) status on 2015-01-06. Fedora 19 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this

Thank you for reporting this bug and we are sorry it could not be fixed.

Comment 24 Edward Rudd 2015-05-07 15:07:10 UTC
This is still an issue in F21/rawhide.. and the patch I attached last will fix the issue.

Comment 25 Javier Celaya 2015-09-28 09:31:26 UTC
I just met this problem with mingw32-nsis-2.46-15.fc22.x86_64 and mingw32-nsis-2.46-16.fc23.x86_64. I can confirm that the patch provided by Edward Rudd fixes the issue.

Note You need to log in before you can comment on or make changes to this bug.