MSYS programs crash when calling fork() on Vista 64. This patch was posted by Oscar Bonilla and modified slightly by me to fix a problem with exceve(). It fixes the crash by filling out a size field in a structure passed to child processes.
This is effectively a duplicate of bug #1674783, which also has a similar patch attached. There are differences, however, particularly in the computation of the size field to be filled in for Vista. Given the level of discussion which has followed the #1674783 bug, I'd tend to favour its associated patch. In any case, the two should be rationalised into a single agreed solution.
I would point out that neither patch could be considered `ready to commit', since neither is accompanied by a properly formatted ChangeLog; this needs to be provided, before any patch can be committed.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
patch for files in msys\rt\src\winsup\cygwin
Logged In: YES
user_id=15438
Originator: NO
You submit patches for reviewing. Once reviewed they can be mared for ready to be committed.
Logged In: YES
user_id=823908
Originator: NO
This is effectively a duplicate of bug #1674783, which also has a similar patch attached. There are differences, however, particularly in the computation of the size field to be filled in for Vista. Given the level of discussion which has followed the #1674783 bug, I'd tend to favour its associated patch. In any case, the two should be rationalised into a single agreed solution.
I would point out that neither patch could be considered `ready to commit', since neither is accompanied by a properly formatted ChangeLog; this needs to be provided, before any patch can be committed.
Logged In: YES
user_id=1369729
Originator: NO
A variant of this patch was accepted into CVS and is included in a recent snapshot.
Thanks,
Cesar