Sanity check pipeline pointer before accessing it. #704

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@Andersbakken
Contributor

I got a crash with this stack:

curl/lib/url.c:2873 (Curl_removeHandleFromPipeline)
curl/lib/url.c:2919 (Curl_getoff_all_pipelines)
curl/lib/multi.c:561 (curl_multi_remove_handle)
curl/lib/url.c:415 (Curl_close)
curl/lib/easy.c:859 (curl_easy_cleanup)

@Andersbakken Andersbakken Sanity check pipeline pointer before accessing it.
I got a crash with this stack:

curl/lib/url.c:2873 (Curl_removeHandleFromPipeline)
curl/lib/url.c:2919 (Curl_getoff_all_pipelines)
curl/lib/multi.c:561 (curl_multi_remove_handle)
curl/lib/url.c:415 (Curl_close)
curl/lib/easy.c:859 (curl_easy_cleanup)
6dd6d6e
@Andersbakken
Contributor

Oops. The first commit had an unintended change in it.

@bagder bagder added a commit that closed this pull request Mar 7, 2016
@Andersbakken @bagder Andersbakken + bagder pipeline: Sanity check pipeline pointer before accessing it.
I got a crash with this stack:

curl/lib/url.c:2873 (Curl_removeHandleFromPipeline)
curl/lib/url.c:2919 (Curl_getoff_all_pipelines)
curl/lib/multi.c:561 (curl_multi_remove_handle)
curl/lib/url.c:415 (Curl_close)
curl/lib/easy.c:859 (curl_easy_cleanup)

Closes #704
46bf7ff
@bagder bagder closed this in 46bf7ff Mar 7, 2016
@bagder
Member
bagder commented Mar 7, 2016

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment