Russell King - ARM Linux wrote:
On Tue, Mar 10, 2009 at 02:13:52PM -0700, Tony Lindgren wrote:
From: Adrian Hunter <[email protected]>
Signed-off-by: Adrian Hunter <[email protected]>
Acked-by: David Brownell <[email protected]>
Signed-off-by: Tony Lindgren <[email protected]>
---
arch/arm/mach-omap2/mmc-twl4030.c | 5 ++++-
arch/arm/mach-omap2/mmc-twl4030.h | 1 +
2 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/mmc-twl4030.c
b/arch/arm/mach-omap2/mmc-twl4030.c
index 9f53d22..9831b2b 100644
--- a/arch/arm/mach-omap2/mmc-twl4030.c
+++ b/arch/arm/mach-omap2/mmc-twl4030.c
@@ -402,7 +402,10 @@ void __init twl4030_mmc_init(struct twl4030_hsmmc_info
*controllers)
return;
}
- sprintf(twl->name, "mmc%islot%i", c->mmc, 1);
+ if (c->name)
+ strncpy(twl->name, c->name, HSMMC_NAME_LEN);
Bug. strncpy can result in a non-null terminated string, and it just so
happens that twl->name has been declared as being HSMMC_NAME_LEN+1 in size.
Well to be pedantic, it is not actually a "bug". twl->name is zero-filled so
HSMMC_NAME_LEN+1
guarantees that it is null terminated.
That's rather unsafe. Using strlcpy() and sizeof(twl->name) instead will
ensure that no matter what happens to the size of the array declaration,
the code remains correct.
Unless the programmer decides to make twl->name a pointer to allocated memory
- then sizeof(twl->name) is no good. You could use ARRAY_SIZE(twl->name) which
(for gcc) includes a check that twl->name is an array.
diff --git a/arch/arm/mach-omap2/mmc-twl4030.h
b/arch/arm/mach-omap2/mmc-twl4030.h
index 0aa1686..ea59e86 100644
--- a/arch/arm/mach-omap2/mmc-twl4030.h
+++ b/arch/arm/mach-omap2/mmc-twl4030.h
@@ -14,6 +14,7 @@ struct twl4030_hsmmc_info {
bool cover_only; /* No card detect - just cover switch */
int gpio_cd; /* or -EINVAL */
int gpio_wp; /* or -EINVAL */
+ char *name; /* or NULL for default */
const?
OK
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
