View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0000657||Rsync||Bug||public||2015-10-29 04:39||2017-11-30 06:09|
|Reporter||DougB||Assigned To||Steven Levine|
|Platform||OS/2||OS||OS/2||OS Version||eCS 2.1|
|Target Version||Fixed in Version|
|Summary||0000657: RSync 3.09 (SHL) and Zip 3.0 (July 5th 2008), crash when they encounter a file that has a blank EA, with no data.|
|Description|| I have encountered a few cases where a file has a blank EA, with no data, in the middle of the EA list (as shown by FM/2 edit Extended Attributes). RSync, and ZIP, both crash when they encounter such a file. See Crash.txt in RSYNCCRASH.zip for details, and a sample file.|
I have not, yet, determined what is actually creating the EAs that cause the problem (see Crash,txt, which describes how I created the file), but neither program should crash when they encounter the problem. It appears that most programs simply drop all of the EAs when they encounter it.
|Steps To Reproduce||See Crash.txt in RSYNCCRASH.zip for details.|
|Additional Information||Product Version was reported as 3.0.6 because that is the closest option to the real version which is 3.0.9 (SHL).|
SHL suggested that I should report this here.
|Tags||No tags attached.|
RSYNCCRASH.zip (25,057 bytes)
||This turns out to be a kernel defect in DosEnumAttribute. Given the broken EAs in the testcase, FEA2 list is not returned as documented, if more than 1 EA is returned. The result is a trap when the code attempts to dereference the oNextEntry pointer. I will update rsync to report this as an error which would have been the case if the kernel had detected the malformed EA data.|
||See also: http://trac.netlabs.org/ports/ticket/73|
It should avoid this exception.
||Okay, I have it. I will try to properly test it in the next few days, and report back. Thanks...|
I haven't been able to create a new defective file, but the fixed version does handle the supplied sample okay.
It seems to me that this should actually be a system wide fix, not just for RSync, but I will leave that up to you. ZIP is still broken, and I need to report that.
Appears to be resolved by
|2015-10-29 04:39||DougB||New Issue|
|2015-10-29 04:39||DougB||File Added: RSYNCCRASH.zip|
|2015-11-04 05:31||Steven Levine||Note Added: 0003067|
|2015-11-04 05:31||Steven Levine||Assigned To||=> Steven Levine|
|2015-11-04 05:31||Steven Levine||Status||new => assigned|
|2015-11-04 05:32||Steven Levine||Note Added: 0003068|
|2017-11-13 02:32||Steven Levine||Status||assigned => feedback|
|2017-11-13 02:32||Steven Levine||Note Added: 0003148|
|2017-11-14 06:19||DougB||Note Added: 0003150|
|2017-11-14 06:19||DougB||Status||feedback => assigned|
|2017-11-17 02:54||DougB||Note Added: 0003151|
|2017-11-30 06:09||Steven Levine||Status||assigned => resolved|
|2017-11-30 06:09||Steven Levine||Resolution||open => fixed|
|2017-11-30 06:09||Steven Levine||Note Added: 0003152|