Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[release/v7.5] Fix up buffer management getting network roots#25311

Closed
pwshBot wants to merge 1 commit into
PowerShell:release/v7.5PowerShell/PowerShell:release/v7.5from
pwshBot:backport/release/v7.5/24600-9418533e8pwshBot/PowerShell:backport/release/v7.5/24600-9418533e8Copy head branch name to clipboard
Closed

[release/v7.5] Fix up buffer management getting network roots#25311
pwshBot wants to merge 1 commit into
PowerShell:release/v7.5PowerShell/PowerShell:release/v7.5from
pwshBot:backport/release/v7.5/24600-9418533e8pwshBot/PowerShell:backport/release/v7.5/24600-9418533e8Copy head branch name to clipboard

Conversation

@pwshBot

@pwshBot pwshBot commented Apr 10, 2025

Copy link
Copy Markdown
Collaborator

Backport of #24600 to release/v7.5

Triggered by @TravisEz13 on behalf of @jborean93

Original CL Label: CL-General

/cc @PowerShell/powershell-maintainers

Impact

Choose either tooling or Customer impact.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Customer Impact

  • Customer reported
  • Found internally

[Select one or both of the boxes. Describe how this issue impacts customers, citing the expected and actual behaviors and scope of the issue. If customer-reported, provide the issue number.]

Regression

  • Yes
  • No

?7.4?

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Testing

[How was the fix verified? How was the issue missed previously? What tests were added?]

Risk

  • High
  • Medium
  • Low

Since PR mentions that there were multiple attempts fix this issue, the backport might be an incomplete fix

[High/Medium/Low. Justify the indication by mentioning how risks were measured and addressed.]

Fixes the logic used when getting the PSDrive DisplayRoot for network PSDrive paths to exclude null terminators.
@TravisEz13 TravisEz13 added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Apr 15, 2025
@TravisEz13

TravisEz13 commented Apr 15, 2025

Copy link
Copy Markdown
Member

@jborean93 Can you confirm this backport includes all the code change to address this issue?

@jborean93

Copy link
Copy Markdown
Collaborator

Seems to include everything from the PR, if the tests are running and it passes then we should be good.

@microsoft-github-policy-service microsoft-github-policy-service Bot added the Review - Needed The PR is being reviewed label Apr 22, 2025
@TravisEz13

Copy link
Copy Markdown
Member

Without a customer impact statement, we are not backporting this 7.5. Please re-open with a customer impact statement if you disagree.

@TravisEz13 TravisEz13 closed this May 14, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Review - Needed The PR is being reviewed label May 14, 2025
@iSazonov

Copy link
Copy Markdown
Collaborator

@TravisEz13 FYI Although the PR is not merged, the original PR has retained the label Backport-7.5.x-Migrated.

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

Labels

CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.