Skip to content

Fix OOB write to S_data in batched linalg_svd (#18625)#18625

Merged
meta-codesync[bot] merged 1 commit intopytorch:mainfrom
abeakkas:export-D99002350
Apr 8, 2026
Merged

Fix OOB write to S_data in batched linalg_svd (#18625)#18625
meta-codesync[bot] merged 1 commit intopytorch:mainfrom
abeakkas:export-D99002350

Conversation

@abeakkas
Copy link
Copy Markdown
Contributor

@abeakkas abeakkas commented Mar 31, 2026

Summary:

S has 3 elements per batch (diagonal of 3x3), but was indexed with the same offset as U and Vh (9 elements per batch), causing out-of-bounds writes and corrupted singular values for batch_size > 1.

Reviewed By: DrJessop

Differential Revision: D99002350

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Mar 31, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/18625

Note: Links to docs will display an error until the docs builds have been completed.

❌ 2 New Failures, 2 Unrelated Failures

As of commit 560aba9 with merge base fcccda3 (image):

NEW FAILURES - The following jobs have failed:

BROKEN TRUNK - The following jobs failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 31, 2026
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented Mar 31, 2026

@abeakkas has exported this pull request. If you are a Meta employee, you can view the originating Diff in D99002350.

@github-actions
Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

Summary:

S has 3 elements per batch (diagonal of 3x3), but was indexed with the same offset as U and Vh (9 elements per batch), causing out-of-bounds writes and corrupted singular values for batch_size > 1.

Reviewed By: DrJessop

Differential Revision: D99002350
@meta-codesync meta-codesync Bot changed the title Fix OOB write to S_data in batched linalg_svd Fix OOB write to S_data in batched linalg_svd (#18625) Apr 7, 2026
@meta-codesync meta-codesync Bot merged commit 4afd7f9 into pytorch:main Apr 8, 2026
159 of 165 checks passed
jpiat pushed a commit to jpiat/executorch that referenced this pull request Apr 14, 2026
Differential Revision: D99002350

Pull Request resolved: pytorch#18625
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants