On Oct 3, 2014, at 6:29 AM, [email protected] wrote:
> Repository: trafficserver
> Updated Branches:
> refs/heads/master 33f651c90 -> d1b3dc66b
>
>
> [TS-3112] - Add null pointer check for contp to prevent core dump after
> handleEvent(TS_FETCH_EVENT_EXT_HEAD_DONE)
>
>
> Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
> Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/d1b3dc66
> Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/d1b3dc66
> Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/d1b3dc66
>
> Branch: refs/heads/master
> Commit: d1b3dc66b5725879949350890ab014cf235cae64
> Parents: 33f651c
> Author: Sudheer Vinukonda <[email protected]>
> Authored: Fri Oct 3 13:29:03 2014 +0000
> Committer: Sudheer Vinukonda <[email protected]>
> Committed: Fri Oct 3 13:29:03 2014 +0000
>
> ----------------------------------------------------------------------
> proxy/FetchSM.cc | 3 +++
> 1 file changed, 3 insertions(+)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/trafficserver/blob/d1b3dc66/proxy/FetchSM.cc
> ----------------------------------------------------------------------
> diff --git a/proxy/FetchSM.cc b/proxy/FetchSM.cc
> index d7b187a..4a79db4 100644
> --- a/proxy/FetchSM.cc
> +++ b/proxy/FetchSM.cc
> @@ -249,6 +249,9 @@ FetchSM::InvokePluginExt(int fetch_event)
> has_sent_header = true;
> }
>
> + if (!contp)
> + goto out;
> +
There's a check for contp being NULL just 10 lines above here ... how can it
become NULL now?
> if (!has_body()) {
> contp->handleEvent(TS_FETCH_EVENT_EXT_BODY_DONE, this);
> goto out;
>
