On Thu, Jan 28, 2021 at 04:25:06PM +0100, Miroslav Lichvar wrote:

> @@ -3038,24 +3042,28 @@ struct port *port_open(const char *phc_device,
>  
>       p->phc_index = phc_index;
>       p->jbod = config_get_int(cfg, interface_name(interface), 
> "boundary_clock_jbod");
> -     transport = config_get_int(cfg, interface_name(interface), 
> "network_transport");
>       p->master_only = config_get_int(cfg, interface_name(interface), 
> "masterOnly");
>       p->bmca = config_get_int(cfg, interface_name(interface), "BMCA");
> +     p->trp = transport_create(cfg, config_get_int(cfg,
> +                           interface_name(interface), "network_transport"));
> +     if (!p->trp) {
> +             goto err_port;
> +     }

The port_open() method just grew a new 'err_log_name' goto label for
rolling back in the case of errors.  I think that will affect this
patch.  May I ask you to rebase and take a look?

Sorry for the churn.

Thanks,
Richard


_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to