Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

applyDiffToFile not checking if the temp file exists before trying to use it #1006

Closed
Pyritie opened this issue Apr 19, 2017 · 5 comments
Closed

Comments

@Pyritie
Copy link

Pyritie commented Apr 19, 2017

DeltaPackage.cs#235.
tempTargetFile is being used but is never checked to see if that file was actually created on disk. In my case, it was not, and so squirrel crashed here.

It had created six folders in SquirrelTemp, tempa through tempf, and it tried to use a tempg that did not exist.

An exception of type 'System.ComponentModel.Win32Exception' occurred in DeltaCompressionDotNet.MsDelta.dll but was not handled in user code

Additional information: The system cannot find the file specified
@Pyritie
Copy link
Author

Pyritie commented Apr 19, 2017

Stack trace:

DeltaCompressionDotNet.MsDelta.dll!DeltaCompressionDotNet.MsDelta.MsDeltaCompression.ApplyDelta(string deltaFilePath, string oldFilePath, string newFilePath)	Unknown
>Squirrel.dll!Squirrel.DeltaPackageBuilder.applyDiffToFile(string deltaPath, string relativeFilePath, string workingDirectory) Line 237	C#
Squirrel.dll!Squirrel.DeltaPackageBuilder.ApplyDeltaPackage.AnonymousMethod__4(string file) Line 120	C#
Squirrel.dll!Squirrel.EnumerableExtensions.ForEach<string>(System.Collections.Generic.IEnumerable<string> source, System.Action<string> onNext) Line 29	C#
Squirrel.dll!Squirrel.DeltaPackageBuilder.ApplyDeltaPackage(Squirrel.ReleasePackage basePackage, Squirrel.ReleasePackage deltaPackage, string outputFile) Line 124	C#
Squirrel.dll!Squirrel.UpdateManager.ApplyReleasesImpl.createFullPackagesFromDeltas.AnonymousMethod__3() Line 331	C#

This happens during await mgr.ApplyReleases(updateInfo).

@gustavocalheiros
Copy link

seems related to this one:
#877

@Thieum
Copy link
Contributor

Thieum commented May 3, 2019

@Pyritie is it an issue you are still experiencing with the latest version of Squirrel?

@Pyritie
Copy link
Author

Pyritie commented May 3, 2019

I haven't had this crash in a long time, no, but I know I made some changes to my project and how I do squirrel releases since then. I can't remember what my setup was that caused this crash.

@Thieum
Copy link
Contributor

Thieum commented May 3, 2019

@shiftkey this can be closed.

@shiftkey shiftkey closed this as completed May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants