Use a memory copy to avoid aliasing when RLP encoding short values

[Amxx]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: 39404fe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
openzeppelin-solidity	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coderabbitai]

Walkthrough

The pull request modifies the encode function in the RLP utility contract. Specifically, in the short-path branch that handles single-byte inputs where the byte value is less than SHORT_OFFSET, the function now returns bytes.concat(input) instead of returning input directly. This modification alters the memory allocation behavior for this specific case while maintaining the same logical output. The long-path execution remains unchanged, and no function signatures were modified.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	No description was provided by the author, making it impossible to assess whether it relates to the changeset.	Add a pull request description explaining the purpose of the memory copy change and why it prevents aliasing issues in RLP encoding.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: using memory copy (via bytes.concat) to avoid memory aliasing in RLP encoding for short values.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ernestognw]

I think it needs a changelog entry, does it?