greybus: uart: Fix the memory leak in connection init

If alloc minor is error, gb_tty should free. Signed-off-by: Phong Tran <[email protected]> Reviewed-by: Viresh Kumar <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
laijs · Jun 4, 2015 · 55f2291 · 55f2291
1 parent 3cb494c
commit 55f2291
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/drivers/staging/greybus/uart.c b/drivers/staging/greybus/uart.c
@@ -618,9 +618,11 @@ static int gb_uart_connection_init(struct gb_connection *connection)
 		if (minor == -ENOSPC) {
 			dev_err(&connection->dev,
 				"no more free minor numbers\n");
-			return -ENODEV;
+			retval = -ENODEV;
+			goto error_version;
 		}
-		return minor;
+		retval = minor;
+		goto error_version;
 	}
 
 	gb_tty->minor = minor;