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

https segfault fix #23

Merged
merged 2 commits into from
Oct 29, 2014
Merged

https segfault fix #23

merged 2 commits into from
Oct 29, 2014

Conversation

sm00th
Copy link
Contributor

@sm00th sm00th commented Oct 10, 2014

Fix for https segfault described in #22

  • Fixed premature error cleanup
  • changed https_fetcher if condition to support https scheme

Fixed premature error cleanup for http fetcher.
Libsoup supports https but restraintd accepted http uris only, changed the
if statement to accept both.
p3ck added a commit that referenced this pull request Oct 29, 2014
@p3ck p3ck merged commit 3c56daf into p3ck:master Oct 29, 2014
@sm00th sm00th deleted the https_segfault branch October 31, 2014 15:13
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

Successfully merging this pull request may close these issues.

2 participants