diff mbox series

[PULL,1/6] linux-user: netlink: Add missing IFA_PROTO to host_to_target_data_addr_rtattr()

Message ID 20241218195247.5459-2-deller@kernel.org (mailing list archive)
State New
Headers show
Series [PULL,1/6] linux-user: netlink: Add missing IFA_PROTO to host_to_target_data_addr_rtattr() | expand

Commit Message

Helge Deller Dec. 18, 2024, 7:52 p.m. UTC
From: Helge Deller <deller@gmx.de>

Fixes this warning:
 Unknown host IFA type: 11

Signed-off-by: Helge Deller <deller@gmx.de>
---
 linux-user/fd-trans.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Laurent Vivier Dec. 24, 2024, 11:57 a.m. UTC | #1
Le 18/12/2024 à 20:52, deller@kernel.org a écrit :
> From: Helge Deller <deller@gmx.de>
> 
> Fixes this warning:
>   Unknown host IFA type: 11
> 
> Signed-off-by: Helge Deller <deller@gmx.de>
> ---
>   linux-user/fd-trans.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/linux-user/fd-trans.c b/linux-user/fd-trans.c
> index c04a97c73a..6191e3115b 100644
> --- a/linux-user/fd-trans.c
> +++ b/linux-user/fd-trans.c
> @@ -1143,6 +1143,7 @@ static abi_long host_to_target_data_addr_rtattr(struct rtattr *rtattr)
>           break;
>       /* string */
>       case IFA_LABEL:
> +    case IFA_PROTO:

IFA_PROTO is NLA_U8, it should be with IFA_ADDRESS and IFA_LOCAL, not IFA_LABEL (it is NLA_STRING).
I agree, it doesn't change anything in the code, but I like to keep same types together.

>           break;
>       /* u32 */
>       case IFA_FLAGS:

Thanks,
Laurent
diff mbox series

Patch

diff --git a/linux-user/fd-trans.c b/linux-user/fd-trans.c
index c04a97c73a..6191e3115b 100644
--- a/linux-user/fd-trans.c
+++ b/linux-user/fd-trans.c
@@ -1143,6 +1143,7 @@  static abi_long host_to_target_data_addr_rtattr(struct rtattr *rtattr)
         break;
     /* string */
     case IFA_LABEL:
+    case IFA_PROTO:
         break;
     /* u32 */
     case IFA_FLAGS: