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.

Sign up for GitHub

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

Jump to bottom

util: fix infinite recursion during inspection #37079

Closed
wants to merge 2 commits into from
Closed

util: fix infinite recursion during inspection #37079

wants to merge 2 commits into from

Conversation

BridgeAR
Copy link
Member

@BridgeAR BridgeAR commented Jan 26, 2021 •
edited
Loading 

doc: fix outdated util inspect documentation and layout example

The documentation and layout example was not up to date with the
latest changes.

util: fix infinite recursion during inspection

A specially crafted object with circular structures behind getters
could cause a infinite recursion. This is now fixed by detecting the
objects as already visited.

Fixes: #37054

@nodejs-github-bot nodejs-github-bot added the util Issues and PRs related to the built-in util module. label Jan 26, 2021
@BridgeAR BridgeAR added the needs-ci PRs that need a full CI run. label Jan 26, 2021
@nodejs-github-bot

This comment has been minimized.

Sign in to view
@Lxxyx Lxxyx removed the needs-ci PRs that need a full CI run. label Jan 27, 2021
aduh95
aduh95 reviewed Jan 29, 2021
View reviewed changes
lib/internal/util/inspect.js Outdated Show resolved Hide resolved
lib/internal/util/inspect.js Outdated Show resolved Hide resolved
Trott
Trott reviewed Jan 31, 2021
View reviewed changes
doc/api/util.md Outdated Show resolved Hide resolved
Trott
Trott approved these changes Jan 31, 2021
View reviewed changes
Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with @aduh95's nits and my question about the doc sample code sorted.

@BridgeAR
Copy link
Member Author

PTAL

@BridgeAR
util: fix infinite recursion during inspection
486f8b8 
A specially crafted object with circular structures behind getters
could cause a infinite recursion. This is now fixed by detecting the
objects as already visited.

Signed-off-by: Ruben Bridgewater <[email protected]>
@BridgeAR
doc: fix outdated util inspect documentation and layout example
7440350 
The documentation and layout example was not up to date with the
latest changes.

Signed-off-by: Ruben Bridgewater <[email protected]>
@BridgeAR BridgeAR requested review from aduh95 and Trott March 22, 2021 00:39
aduh95
aduh95 approved these changes Mar 22, 2021
View reviewed changes
Copy link
Contributor

@aduh95 aduh95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

@antsmartian antsmartian added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 29, 2021
@nodejs-github-bot

This comment has been minimized.

Sign in to view
@nodejs-github-bot

This comment has been minimized.

Sign in to view
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Apr 30, 2021 •
edited by jasnell
Loading

CI: https://ci.nodejs.org/job/node-test-pull-request/37845/ 💚

@jasnell
Copy link
Member

jasnell commented Apr 30, 2021

Landed in 3c6ff9e...5bfb6f0

@jasnell jasnell closed this Apr 30, 2021
jasnell pushed a commit that referenced this pull request Apr 30, 2021
@BridgeAR @jasnell
util: fix infinite recursion during inspection
942b76d 
A specially crafted object with circular structures behind getters
could cause a infinite recursion. This is now fixed by detecting the
objects as already visited.

Signed-off-by: Ruben Bridgewater <[email protected]>

PR-URL: #37079
Fixes: #37054
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
jasnell pushed a commit that referenced this pull request Apr 30, 2021
@BridgeAR @jasnell
doc: fix outdated util inspect documentation and layout example
5bfb6f0 
The documentation and layout example was not up to date with the
latest changes.

Signed-off-by: Ruben Bridgewater <[email protected]>

PR-URL: #37079
Fixes: #37054
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
targos pushed a commit that referenced this pull request May 3, 2021
@BridgeAR @targos
util: fix infinite recursion during inspection
ec5b06e 
A specially crafted object with circular structures behind getters
could cause a infinite recursion. This is now fixed by detecting the
objects as already visited.

Signed-off-by: Ruben Bridgewater <[email protected]>

PR-URL: #37079
Fixes: #37054
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
targos pushed a commit that referenced this pull request May 3, 2021
@BridgeAR @targos
doc: fix outdated util inspect documentation and layout example
9258604 
The documentation and layout example was not up to date with the
latest changes.

Signed-off-by: Ruben Bridgewater <[email protected]>

PR-URL: #37079
Fixes: #37054
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
@targos targos mentioned this pull request May 3, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juanarbol juanarbol juanarbol approved these changes

@trivikr trivikr trivikr approved these changes

@aduh95 aduh95 aduh95 approved these changes

@antsmartian antsmartian antsmartian approved these changes

@Trott Trott Trott approved these changes

@jasnell jasnell jasnell approved these changes

Assignees
No one assigned
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. util Issues and PRs related to the built-in util module.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

specific scenario causes infinite recursion in util.inspect (max stack size exceeded)
10 participants
@BridgeAR @nodejs-github-bot @jasnell @Trott @antsmartian @aduh95 @trivikr @juanarbol @targos @Lxxyx