> Date: Wed, 11 Mar 2020 13:38:12 +0100
> From: Jasper Lievisse Adriaanse <jas...@openbsd.org>
> 
> Hi,
> 
> Remove dead code which is actually duplicated a few lines above
> right after err is set. Coverity ID 975917
> 
> OK?

ok kettenis@

> Index: dev/usb/uplcom.c
> ===================================================================
> RCS file: /cvs/src/sys/dev/usb/uplcom.c,v
> retrieving revision 1.73
> diff -u -p -U11 -r1.73 uplcom.c
> --- dev/usb/uplcom.c  18 Nov 2018 16:23:14 -0000      1.73
> +++ dev/usb/uplcom.c  11 Mar 2020 12:36:03 -0000
> @@ -619,27 +619,22 @@ uplcom_param(void *addr, int portno, str
>       if (err) {
>               DPRINTF(("uplcom_param: err=%s\n", usbd_errstr(err)));
>               return (EIO);
>       }
>  
>       if (ISSET(t->c_cflag, CRTSCTS))
>               uplcom_set_crtscts(sc);
>  
>       if (sc->sc_rts == -1 || sc->sc_dtr == -1)
>               uplcom_set_line_state(sc);
>  
> -     if (err) {
> -             DPRINTF(("uplcom_param: err=%s\n", usbd_errstr(err)));
> -             return (EIO);
> -     }
> -
>       return (0);
>  }
>  
>  int
>  uplcom_open(void *addr, int portno)
>  {
>       struct uplcom_softc *sc = addr;
>       usb_device_request_t req;
>       usbd_status uerr;
>       int err;
>  
> 
> -- 
> jasper
> 
> 

Reply via email to