[ovs-dev] [PATCH ovn v2] chassis.c: Fix the possible NULL pointer deference in chassis_cleanup().
Numan Siddique
numans at ovn.org
Mon Sep 14 19:03:49 UTC 2020
On Sat, Sep 12, 2020 at 11:19 AM Han Zhou <hzhou at ovn.org> wrote:
> If chassis_rec is NULL but chassis_private_rec is not, chassis_rec->name
> is a NULL pointer deference. This patch fixes it.
>
> Fixes: 4adc10f581 ("Avoid nb_cfg update notification flooding")
> Signed-off-by: Han Zhou <hzhou at ovn.org>
>
Acked-by: Numan Siddique <numans at ovn.org>
Numan
> ---
> v1 -> v2: Address the comment from Ilya for coding style.
>
> controller/chassis.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/controller/chassis.c b/controller/chassis.c
> index 8e6ad2d..a365188 100644
> --- a/controller/chassis.c
> +++ b/controller/chassis.c
> @@ -843,7 +843,8 @@ chassis_cleanup(struct ovsdb_idl_txn *ovnsb_idl_txn,
> if (ovnsb_idl_txn) {
> ovsdb_idl_txn_add_comment(ovnsb_idl_txn,
> "ovn-controller: unregistering chassis
> '%s'",
> - chassis_rec->name);
> + chassis_rec ? chassis_rec->name
> + : chassis_private_rec->name);
> if (chassis_rec) {
> sbrec_chassis_delete(chassis_rec);
> }
> --
> 2.1.0
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
More information about the dev
mailing list