Skip to content

[v6.x backport] src: free memory before re-setting URLHost value - #19639

Closed
prog1dev wants to merge 1 commit into
nodejs:v6.x-stagingfrom
prog1dev:backport-18357-to-v6.x
Closed

[v6.x backport] src: free memory before re-setting URLHost value#19639
prog1dev wants to merge 1 commit into
nodejs:v6.x-stagingfrom
prog1dev:backport-18357-to-v6.x

Conversation

@prog1dev

@prog1devprog1dev commented Mar 27, 2018

Copy link
Copy Markdown
Contributor

Original pr #18357

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@MylesBorins

@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. dont-land-on-v4.x whatwg-url Issues and PRs related to the WHATWG URL implementation. labels Mar 27, 2018
@prog1dev

prog1dev commented Mar 29, 2018

Copy link
Copy Markdown
ContributorAuthor

Something went wrong 😄
+848 −38,000

@TimothyGu

Copy link
Copy Markdown
Member

@prog1dev It’s probably because there have been force-pushes to the v8.x-staging branch. A careful rebase should fix the problem.

Fixes: nodejs#18302
PR-URL: nodejs#18357
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@prog1dev
prog1devforce-pushed the backport-18357-to-v6.x branch from f067901 to af2c41eCompareApril 1, 2018 12:42
@MylesBorins

Copy link
Copy Markdown
Contributor

MylesBorins pushed a commit that referenced this pull request Apr 13, 2018
Fixes: #18302
Backport-PR-URL: #19639
PR-URL: #18357
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@MylesBorins

Copy link
Copy Markdown
Contributor

landed in 0ca2dad

@prog1dev
prog1dev deleted the backport-18357-to-v6.x branch April 13, 2018 09:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.whatwg-urlIssues and PRs related to the WHATWG URL implementation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@prog1dev@TimothyGu@MylesBorins@nodejs-github-bot