[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH 3/6] xennet: fix IN annotations


  • To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
  • From: Césaire Mounah <cesaire.mounah@xxxxxxxxxx>
  • Date: Mon, 5 Oct 2026 18:03:28 +0200
  • Authentication-results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=vates.tech header.i="@vates.tech" header.h="From:Subject:Date:Message-ID:To:Cc:MIME-Version:Content-Type:In-Reply-To:References:Feedback-ID"
  • Cc: Césaire Mounah <cesaire.mounah@xxxxxxxxxx>
  • Delivery-date: Mon, 05 Oct 2026 16:03:45 +0000
  • Feedback-id: default:8631fc262581453bbf619ec5b2062170:Sweego
  • List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>

* string.c: FormatNumber only writes to Buffer, so it should be
annotated as _Out_

* adapter.c: __AdapterMultiSzToUpcaseAnsi reads and writes to Buffer param
so it should be annotated as _Inout_

* adapter.c: __AdapterMatchDistribution calls util.c::__strtork_r(buffer, ...)
that is annotated as (_Inout_opt_, _In_, _Inout_) so buffer paramater should
be annotated as _Inout_opt_.

Signed-off-by: Césaire Mounah <cesaire.mounah@xxxxxxxxxx>
---
 src/xennet/adapter.c | 4 ++--
 src/xennet/string.c  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/xennet/adapter.c b/src/xennet/adapter.c
index ba7fd2a..9fd93d3 100644
--- a/src/xennet/adapter.c
+++ b/src/xennet/adapter.c
@@ -1542,7 +1542,7 @@ __AdapterFree(
 
 static FORCEINLINE PANSI_STRING
 __AdapterMultiSzToUpcaseAnsi(
-    _In_ PCHAR      Buffer
+    _Inout_ PCHAR   Buffer
     )
 {
     PANSI_STRING    Ansi;
@@ -1621,7 +1621,7 @@ __AdapterFreeAnsi(
 static FORCEINLINE BOOLEAN
 __AdapterMatchDistribution(
     _In_ PXENNET_ADAPTER    Adapter,
-    _In_ PCHAR              Buffer
+    _Inout_opt_ PCHAR       Buffer
     )
 {
     PCHAR                   Vendor;
diff --git a/src/xennet/string.c b/src/xennet/string.c
index b75e249..a740951 100644
--- a/src/xennet/string.c
+++ b/src/xennet/string.c
@@ -53,7 +53,7 @@ __StringPut(
 
 static PCHAR
 FormatNumber(
-    _In_ PCHAR      Buffer,
+    _Out_ PCHAR     Buffer,
     _In_ ULONGLONG  Value,
     _In_ UCHAR      Base,
     _In_ BOOLEAN    UpperCase
-- 
2.55.0.windows.5



--
Césaire Mounah | Vates Windows Guest Tools Engineer

XCP-ng & Xen Orchestra - Vates solutions

web: https://vates.tech

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.