more

microsoft · Nov 30, 2023 · 49d692e · 49d692e
1 parent aa9e945
commit 49d692e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/xdpapi/xdpapi.c b/src/xdpapi/xdpapi.c
@@ -172,7 +172,7 @@ XdpInterfaceOpen(
 HRESULT
 XdpRssGetCapabilities(
     _In_ HANDLE InterfaceHandle,
-    _Out_opt_ XDP_RSS_CAPABILITIES *RssCapabilities,
+    _Out_writes_bytes_opt_(*RssCapabilitiesSize) XDP_RSS_CAPABILITIES *RssCapabilities,
     _Inout_ UINT32 *RssCapabilitiesSize
     )
 {
@@ -210,7 +210,7 @@ XdpRssSet(
 HRESULT
 XdpRssGet(
     _In_ HANDLE InterfaceHandle,
-    _Out_opt_ XDP_RSS_CONFIGURATION *RssConfiguration,
+    _Out_writes_bytes_opt_(*RssConfigurationSize) XDP_RSS_CONFIGURATION *RssConfiguration,
     _Inout_ UINT32 *RssConfigurationSize
     )
 {

diff --git a/test/rssconfig/rssconfig.c b/test/rssconfig/rssconfig.c
@@ -202,7 +202,7 @@ ProcessCommandGet(
 
     RssConfigSize = 0;
     Result = XdpRssGet(InterfaceHandle, RssConfig, &RssConfigSize);
-    if (SUCCEEDED(Result) || RssConfigSize < sizeof(*RssConfig)) {
+    if (SUCCEEDED(Result)) {
         printf(
             "Error: Failed to get RSS configuration size on IfIndex=%u Result=%d RssConfigSize=%d\n",
             IfIndex, Result, RssConfigSize);
@@ -215,7 +215,6 @@ ProcessCommandGet(
         goto Exit;
     }
 
-    _Analysis_assume_(RssConfigSize >= sizeof(*RssConfig));
     Result = XdpRssGet(InterfaceHandle, RssConfig, &RssConfigSize);
     if (FAILED(Result)) {
         printf("Error: Failed to get RSS configuration on IfIndex=%u Result=%d\n", IfIndex, Result);