Patch "driver core: dd: fix return value of __setup handler" has been added to the 5.15-stable tree

[Sasha Levin <sashal@xxxxxxxxxx>]

This is a note to let you know that I've just added the patch titled

    driver core: dd: fix return value of __setup handler

to the 5.15-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     driver-core-dd-fix-return-value-of-__setup-handler.patch
and it can be found in the queue-5.15 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 9f1fb57b0613b630413ee22b4a5f1bf0e67bb692
Author: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
Date:   Mon Feb 28 20:18:29 2022 -0800

    driver core: dd: fix return value of __setup handler
    
    [ Upstream commit f2aad54703dbe630f9d8b235eb58e8c8cc78f37d ]
    
    When "driver_async_probe=nulltty" is used on the kernel boot command line,
    it causes an Unknown parameter message and the string is added to init's
    environment strings, polluting them.
    
      Unknown kernel command line parameters "BOOT_IMAGE=/boot/bzImage-517rc6
      driver_async_probe=nulltty", will be passed to user space.
    
     Run /sbin/init as init process
       with arguments:
         /sbin/init
       with environment:
         HOME=/
         TERM=linux
         BOOT_IMAGE=/boot/bzImage-517rc6
         driver_async_probe=nulltty
    
    Change the return value of the __setup function to 1 to indicate
    that the __setup option has been handled.
    
    Link: lore.kernel.org/r/64644a2f-4a20-bab3-1e15-3b2cdd0defe3@xxxxxxxxxxxx
    Fixes: 1ea61b68d0f8 ("async: Add cmdline option to specify drivers to be async probed")
    Cc: Feng Tang <feng.tang@xxxxxxxxx>
    Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
    Cc: "Rafael J. Wysocki" <rafael@xxxxxxxxxx>
    Reported-by: Igor Zhbanov <i.zhbanov@xxxxxxxxxxxx>
    Reviewed-by: Feng Tang <feng.tang@xxxxxxxxx>
    Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
    Link: https://lore.kernel.org/r/20220301041829.15137-1-rdunlap@xxxxxxxxxxxxx
    Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/drivers/base/dd.c b/drivers/base/dd.c
index 6b6630693201..64ce42b6c6b6 100644
--- a/drivers/base/dd.c
+++ b/drivers/base/dd.c
@@ -809,7 +809,7 @@ static int __init save_async_options(char *buf)
 		pr_warn("Too long list of driver names for 'driver_async_probe'!\n");
 
 	strlcpy(async_probe_drv_names, buf, ASYNC_DRV_NAMES_MAX_LEN);
-	return 0;
+	return 1;
 }
 __setup("driver_async_probe=", save_async_options);