{"sha":"ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd","node_id":"C_kwDOES-Pt9oAKGNhNjVkNDRlNzQzNDI2N2VmZjdhOTVkMGQyZTljNmFiNjNhZDE4ZGQ","commit":{"author":{"name":"Aniruddha Kanhere","email":"kanherea@amazon.com","date":"2026-04-13T17:47:20Z"},"committer":{"name":"Archit Gupta","email":"archigup@amazon.com","date":"2026-04-29T00:35:41Z"},"message":"Fix DHCPv6 sub-option default case skip\n\nIn prvDHCPv6_subOption(), the default case for unknown sub-option types\nskipped uxLength2 - uxUsed bytes. Since uxUsed is cumulative bytes from\nthe parent option start and uxLength2 is the current sub-option payload\nlength, this subtraction wraps when uxUsed > uxLength2, corrupting the\nparser index. Fix by skipping uxLength2 bytes. Also add xHasError check\nto the loop exit condition.","tree":{"sha":"9c1dcfaeb254dba8b5dd6d12c5501bdf9c8ba025","url":"https://api.github.com/repos/FreeRTOS/FreeRTOS-Plus-TCP/git/trees/9c1dcfaeb254dba8b5dd6d12c5501bdf9c8ba025"},"url":"https://api.github.com/repos/FreeRTOS/FreeRTOS-Plus-TCP/git/commits/ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd","comment_count":0,"verification":{"verified":false,"reason":"unsigned","signature":null,"payload":null,"verified_at":null}},"url":"https://api.github.com/repos/FreeRTOS/FreeRTOS-Plus-TCP/commits/ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd","html_url":"https://github.com/FreeRTOS/FreeRTOS-Plus-TCP/commit/ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd","comments_url":"https://api.github.com/repos/FreeRTOS/FreeRTOS-Plus-TCP/commits/ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd/comments","author":{"login":"AniruddhaKanhere","id":60444055,"node_id":"MDQ6VXNlcjYwNDQ0MDU1","avatar_url":"https://avatars.githubusercontent.com/u/60444055?v=4","gravatar_id":"","url":"https://api.github.com/users/AniruddhaKanhere","html_url":"https://github.com/AniruddhaKanhere","followers_url":"https://api.github.com/users/AniruddhaKanhere/followers","following_url":"https://api.github.com/users/AniruddhaKanhere/following{/other_user}","gists_url":"https://api.github.com/users/AniruddhaKanhere/gists{/gist_id}","starred_url":"https://api.github.com/users/AniruddhaKanhere/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AniruddhaKanhere/subscriptions","organizations_url":"https://api.github.com/users/AniruddhaKanhere/orgs","repos_url":"https://api.github.com/users/AniruddhaKanhere/repos","events_url":"https://api.github.com/users/AniruddhaKanhere/events{/privacy}","received_events_url":"https://api.github.com/users/AniruddhaKanhere/received_events","type":"User","user_view_type":"public","site_admin":false},"committer":{"login":"archigup","id":71798289,"node_id":"MDQ6VXNlcjcxNzk4Mjg5","avatar_url":"https://avatars.githubusercontent.com/u/71798289?v=4","gravatar_id":"","url":"https://api.github.com/users/archigup","html_url":"https://github.com/archigup","followers_url":"https://api.github.com/users/archigup/followers","following_url":"https://api.github.com/users/archigup/following{/other_user}","gists_url":"https://api.github.com/users/archigup/gists{/gist_id}","starred_url":"https://api.github.com/users/archigup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/archigup/subscriptions","organizations_url":"https://api.github.com/users/archigup/orgs","repos_url":"https://api.github.com/users/archigup/repos","events_url":"https://api.github.com/users/archigup/events{/privacy}","received_events_url":"https://api.github.com/users/archigup/received_events","type":"User","user_view_type":"public","site_admin":false},"parents":[{"sha":"fe123ba5c7ed9591cc87f44e234eb2b63ccf5a81","url":"https://api.github.com/repos/FreeRTOS/FreeRTOS-Plus-TCP/commits/fe123ba5c7ed9591cc87f44e234eb2b63ccf5a81","html_url":"https://github.com/FreeRTOS/FreeRTOS-Plus-TCP/commit/fe123ba5c7ed9591cc87f44e234eb2b63ccf5a81"}],"stats":{"total":5,"additions":3,"deletions":2},"files":[{"sha":"e66f70ea32a01e127da6cfa154c4ce5e8b0877a5","filename":"source/FreeRTOS_DHCPv6.c","status":"modified","additions":3,"deletions":2,"changes":5,"blob_url":"https://github.com/FreeRTOS/FreeRTOS-Plus-TCP/blob/ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd/source%2FFreeRTOS_DHCPv6.c","raw_url":"https://github.com/FreeRTOS/FreeRTOS-Plus-TCP/raw/ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd/source%2FFreeRTOS_DHCPv6.c","contents_url":"https://api.github.com/repos/FreeRTOS/FreeRTOS-Plus-TCP/contents/source%2FFreeRTOS_DHCPv6.c?ref=ca65d44e7434267eff7a95d0d2e9c6ab63ad18dd","patch":"@@ -1191,13 +1191,14 @@ static BaseType_t prvDHCPv6_subOption( uint16_t usOption,\n                 break;\n \n             default:\n-                ( void ) xBitConfig_read_uc( pxMessage, NULL, uxLength2 - uxUsed );\n+                ( void ) xBitConfig_read_uc( pxMessage, NULL, uxLength2 );\n                 FreeRTOS_printf( ( \"prvDHCPv6Analyse: skipped unknown option %u\\n\", usOption2 ) );\n                 break;\n         }\n \n-        if( xReturn != pdTRUE )\n+        if( ( xReturn != pdTRUE ) || ( pxMessage->xHasError != pdFALSE ) )\n         {\n+            xReturn = pdFALSE;\n             FreeRTOS_printf( ( \"prvDHCPv6_subOption: One of sub-options %d returns error\\n\", usOption2 ) );\n             break;\n         }"}]}