Skip to content

Commit fe579e8

Browse files
generatedunixname537391475639613facebook-github-bot
generatedunixname537391475639613
authored andcommitted
Fix for T219433414 ("Your diff, D69161451, broke some tests")
Summary: This diff was created by the [Devmate](https://fb.workplace.com/groups/1205375940786433/permalink/1205378807452813/) to automatically fix the broken tests assigned to you / your team in T219433414: - The agent searched for and edited the files it deemed relevant for fixing the tests - After generating the patch, the broken test(s) were rerun to verify they now pass - CI tests must pass in Phabricator before this diff is published; otherwise, it is in draft state and should not be reviewed --- **If this diff is the correct fix for T219433414, please accept and land the diff**. *Note that while we set a high quality bar for the agent publishing these diffs, LLM-powered agents still make mistakes, so your critical review is much appreciated and will help us improve!* **If changes need to be made:** Please commandeer the diff, or comment with the specific changes needed. --- *P.S. If you have broader feedback, please post in the [Devmate - Users group](https://fb.workplace.com/groups/1205375940786433)! We would love to hear from you on improving the agent to better assist you in your daily work.* Reviewed By: afrind Differential Revision: D72030656 fbshipit-source-id: ee3b6d8d5410f3dd1c7e251bf6754b61685f3f8a
1 parent 91d4eda commit fe579e8

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

proxygen/lib/http/codec/HTTP2Framer.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -460,6 +460,9 @@ ErrorCode parseRFC9218Priority(Cursor& cursor,
460460
if (header.stream != 0) {
461461
return ErrorCode::PROTOCOL_ERROR;
462462
}
463+
if (header.length <= sizeof(priStream)) {
464+
return ErrorCode::FRAME_SIZE_ERROR;
465+
}
463466
priStream = parseUint31(cursor);
464467
auto priLen = header.length - sizeof(priStream);
465468
outPriority.resize(priLen);

0 commit comments

Comments
 (0)