s390/pci: avoid using strncmp with hardcoded length
authorVasily Gorbik <gor@linux.ibm.com>
Mon, 19 Aug 2019 14:29:01 +0000 (16:29 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Thu, 29 Aug 2019 13:34:57 +0000 (15:34 +0200)
Command line option values passed to __setup callbacks are always
null-terminated and "s390_iommu=" may only accept "strict" as value.
So replace strncmp with strcmp.

While at it also make s390_iommu_setup return 1, which means this
command line option is handled by this callback.

Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
arch/s390/pci/pci_dma.c

index 9e52d1527f71495f920a6d39989fb515410f3056..fb2c7db0164e6cbce83f80eeec975d3006d1e6bc 100644 (file)
@@ -674,9 +674,9 @@ EXPORT_SYMBOL_GPL(s390_pci_dma_ops);
 
 static int __init s390_iommu_setup(char *str)
 {
-       if (!strncmp(str, "strict", 6))
+       if (!strcmp(str, "strict"))
                s390_iommu_strict = 1;
-       return 0;
+       return 1;
 }
 
 __setup("s390_iommu=", s390_iommu_setup);