From: Wei Yongjun <weiyongj...@huawei.com>

[ Upstream commit ff62255a2a5c1228a28f2bb063646f948115a309 ]

Fix to return negative error code -ENOMEM from the error handling
case instead of 0, as done elsewhere in this function.

Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
Link: https://lore.kernel.org/r/20200427122415.47416-1-weiyongj...@huawei.com
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 drivers/tty/vcc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/tty/vcc.c b/drivers/tty/vcc.c
index d2a1e1228c82d..9ffd42e333b83 100644
--- a/drivers/tty/vcc.c
+++ b/drivers/tty/vcc.c
@@ -605,6 +605,7 @@ static int vcc_probe(struct vio_dev *vdev, const struct 
vio_device_id *id)
        port->index = vcc_table_add(port);
        if (port->index == -1) {
                pr_err("VCC: no more TTY indices left for allocation\n");
+               rv = -ENOMEM;
                goto free_ldc;
        }
 
-- 
2.25.1



Reply via email to