[PATCH] Fix dediprog crash if usb_open() fails

[David Woodhouse]

Signed-off-by: David Woodhouse

---
Ick. You made me touch Subversion, even if only through 'git svn clone'.

diff --git a/dediprog.c b/dediprog.c
index fb95d10..2640808 100644
--- a/dediprog.c
+++ b/dediprog.c
@@ -884,6 +884,10 @@ int dediprog_init(void)
    msg_pdbg("Found USB device (%04x:%04x).\n",
    	 dev->descriptor.idVendor, dev->descriptor.idProduct);
    dediprog_handle = usb_open(dev);
+	if (!dediprog_handle) {
+		msg_perr("Could not open USB device: %s\n", usb_strerror());
+		return 1;
+	}
    ret = usb_set_configuration(dediprog_handle, 1);
    if (ret < 0) {
    	msg_perr("Could not set USB device configuration: %i %s\n",

[Stefan Tauner]

On Mon, 29 Jul 2013 14:40:14 +0100
David Woodhouse  wrote:
> Signed-off-by: David Woodhouse
>
> Ick. You made me touch Subversion, even if only through 'git svn clone'.

Just to remind you how good we have it today ;)
> diff --git a/dediprog.c b/dediprog.c
> index fb95d10..2640808 100644
> --- a/dediprog.c
> +++ b/dediprog.c
> @@ -884,6 +884,10 @@ int dediprog_init(void)
>    msg_pdbg("Found USB device (%04x:%04x).\n",
>    	 dev->descriptor.idVendor, dev->descriptor.idProduct);
>    dediprog_handle = usb_open(dev);
> if (!dediprog_handle) {
> msg_perr("Could not open USB device: %s\n", usb_strerror());
> return 1;
> }
> ret = usb_set_configuration(dediprog_handle, 1);
> if (ret < 0) {
> msg_perr("Could not set USB device configuration: %i %s\n",

Excellent, thank you!
Acked-by: Stefan Tauner 
and committed in r1705.