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

Address some missed shred calls #27801

Merged
merged 3 commits into from
Jun 26, 2018
Merged

Address some missed shred calls #27801

merged 3 commits into from
Jun 26, 2018

Conversation

omus
Copy link
Member

@omus omus commented Jun 26, 2018

Discovered some missing shred calls when using Julia 0.7-beta. These were noticed by using pkg up with a repo requiring credentials and reaching the prompt abort condition.

Introduced in #27565

@omus omus requested a review from mbauman June 26, 2018 16:39
@omus omus added libgit2 The libgit2 library or the LibGit2 stdlib module security System security concerns and vulnerabilities labels Jun 26, 2018
@omus omus added this to the 0.7 milestone Jun 26, 2018
@ararslan
Copy link
Member

Not sure whether it's related, but FreeBSD seems to be freezing on the LibGit2 tests. I'll see if I can reproduce locally.

@ararslan
Copy link
Member

Worked okay locally.

@ararslan
Copy link
Member

FreeBSD failure is unreproducible, macOS failure is the FileWatching thing.

@ararslan ararslan merged commit bec49b0 into master Jun 26, 2018
@ararslan ararslan deleted the cv/missed-shred branch June 26, 2018 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libgit2 The libgit2 library or the LibGit2 stdlib module security System security concerns and vulnerabilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants