.78 & .79 crash when processing attached bin file #109

Closed
opened 2026-01-29 16:35:30 +00:00 by claunia · 4 comments
Owner

Originally created by @DHLgh on GitHub (Feb 7, 2016).

Originally assigned to: @canihavesomecoffee on GitHub.

This issue began with version .78 and is still present in .79. Thank you!
ProblemFile.zip

Originally created by @DHLgh on GitHub (Feb 7, 2016). Originally assigned to: @canihavesomecoffee on GitHub. This issue began with version .78 and is still present in .79. Thank you! [ProblemFile.zip](https://github.com/CCExtractor/ccextractor/files/120846/ProblemFile.zip)
Author
Owner

@barun511 commented on GitHub (Mar 12, 2016):

Unable to reproduce, running ccextractor returns the attached file (turned into a .txt to comply with git)
problemfileresult.txt

@barun511 commented on GitHub (Mar 12, 2016): Unable to reproduce, running ccextractor returns the attached file (turned into a .txt to comply with git) [problemfileresult.txt](https://github.com/CCExtractor/ccextractor/files/170548/problemfileresult.txt)
Author
Owner

@anshul1912 commented on GitHub (Mar 12, 2016):

Already solved by arbitrer, only reproducible on Windows

@anshul1912 commented on GitHub (Mar 12, 2016): Already solved by arbitrer, only reproducible on Windows
Author
Owner

@cfsmp3 commented on GitHub (Mar 12, 2016):

But is this solution good? Isn't it the one that checked if the file was
open instead of the reason for which it could be closed?

On Sat, Mar 12, 2016 at 8:13 PM, Anshul Maheshwari <notifications@github.com

wrote:

Already solved by arbitrer, only reproducible on Windows


Reply to this email directly or view it on GitHub
https://github.com/CCExtractor/ccextractor/issues/277#issuecomment-195793194
.

@cfsmp3 commented on GitHub (Mar 12, 2016): But is this solution good? Isn't it the one that checked if the file was open instead of the reason for which it could be closed? On Sat, Mar 12, 2016 at 8:13 PM, Anshul Maheshwari <notifications@github.com > wrote: > > Already solved by arbitrer, only reproducible on Windows > > — > Reply to this email directly or view it on GitHub > https://github.com/CCExtractor/ccextractor/issues/277#issuecomment-195793194 > .
Author
Owner

@canihavesomecoffee commented on GitHub (May 22, 2016):

Can be closed once #323 is merged (after bot confirms that PR doesn't introduce new regressions). Tested the PR manually and it solves this issue.

@canihavesomecoffee commented on GitHub (May 22, 2016): Can be closed once #323 is merged (after bot confirms that PR doesn't introduce new regressions). Tested the PR manually and it solves this issue.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#109