Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

http: incoming parser set to null twice causes crash #4948

Closed
wants to merge 2 commits into from
Closed

http: incoming parser set to null twice causes crash #4948

wants to merge 2 commits into from

Commits on Mar 8, 2013

  1. http: check if incoming parser has already been freed 

    Fix nodejs#4948

This adds a check before setting the incoming parser
to null. Under certain circumstances it'll already be set to
null by freeParser().

Otherwise this will cause node to crash as it tries to set
null on something that is already null.
    @hnry
    hnry committed Mar 8, 2013
    Configuration menu
    Copy the full SHA
    73380f5 View commit details
    Browse the repository at this point in the history

Commits on Mar 9, 2013

  1. regression test for nodejs#4948 

    this test crashes as it'll try to set the incoming parser
to null twice
    @hnry
    hnry committed Mar 9, 2013
    Configuration menu
    Copy the full SHA
    2ba2b7d View commit details
    Browse the repository at this point in the history