Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Commit f337882

Browse files
committed
Addressed PR feedback
Cf. #26086 (comment)
1 parent d2f5b2c commit f337882

File tree

1 file changed

+2
-1
lines changed
  • src/System.Collections/src/System/Collections/Generic

1 file changed

+2
-1
lines changed

src/System.Collections/src/System/Collections/Generic/Stack.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,8 @@ public T Pop()
235235
T[] array = _array;
236236

237237
// if (_size == 0) is equivalent to if (size == -1), and this case
238-
// is covered with (uint)size, thus allowing RCE https://github.com/dotnet/coreclr/pull/9773
238+
// is covered with (uint)size, thus allowing bounds check elimination
239+
// https://github.com/dotnet/coreclr/pull/9773
239240
if ((uint)size >= (uint)array.Length)
240241
{
241242
ThrowForEmptyStack();

0 commit comments

Comments
 (0)