On 2/3/25 2:33 PM, Dumitru Ceara wrote:
>> +            struct re_nl_received_route_node *rr = xzalloc(sizeof *rr);
> Nit: xmalloc() is good enough.
> 

I see now that in patch 7/11 we implicitly rely on the ifname (added in
that patch) to be zeroed.  I still think we should keep xmalloc() here
and just add the null terminator in the next patch.

>> +            ovs_list_push_back(handle_data->learned_routes, &rr->list_node);
> Nit: I'd push this to the list after it is fully initialized.
> 
>> +            rr->db = handle_data->db;
>> +            rr->addr = rd->rta_dst;
>> +            rr->plen = rd->rtm_dst_len;
>> +            rr->nexthop = nexthop->addr;
>> +        }


_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to