lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Mon, 19 Sep 2022 15:20:53 +0200
From:   Eelco Chaudron <>
To:     Nathan Huckleberry <>
Cc:     Dan Carpenter <>,,
        Pravin B Shelar <>,
        "David S. Miller" <>,
        Eric Dumazet <>,
        Jakub Kicinski <>,
        Paolo Abeni <>,
        Nathan Chancellor <>,
        Nick Desaulniers <>,
        Tom Rix <>,,,
Subject: Re: [PATCH] openvswitch: Change the return type for vport_ops.send
 function hook to int

On 14 Sep 2022, at 1:07, Nathan Huckleberry wrote:

> All usages of the vport_ops struct have the .send field set to
> dev_queue_xmit or internal_dev_recv.  Since most usages are set to
> dev_queue_xmit, the function hook should match the signature of
> dev_queue_xmit.
> The only call to vport_ops->send() is in net/openvswitch/vport.c and it
> throws away the return value.
> This mismatched return type breaks forward edge kCFI since the underlying
> function definition does not match the function hook definition.
> Reported-by: Dan Carpenter <>
> Link:
> Cc:
> Signed-off-by: Nathan Huckleberry <>

The changes look good to me.

Acked-by: Eelco Chaudron <>

Powered by blists - more mailing lists