On Tuesday 11 March 2008, Len Brown wrote:
> On Thursday 06 March 2008, Johann Felix Soden wrote:
> > From: Johann Felix Soden <johfel@xxxxxxxxxxxxxxxxxxxxx>
> >
> > Since "ff_gbl_lock" has a length of 10 chars and is copied with sprintf to
> > char buffer[10], there is a problem because of the closing zero byte. We
> > need char buffer[11].
>
> "ff_gbl_lock" is 11 characters, and sprintf adds a null, so
> buffer should be 12, yes?
I have noticed my miscounting and have sent a new patch to you, Len
Brown, some days ago. If it is needed anymore, below is the corrected
patch again.
---
From: Johann Felix Soden <johfel@xxxxxxxxxxxxxxxxxxxxx>
Since "ff_gbl_lock" has a length of 11 chars and is copied with sprintf
to char buffer[10], there is a problem. We need char buffer[12] because
of the closing zero byte.
Signed-off-by: Johann Felix Soden <johfel@xxxxxxxxxxxxxxxxxxxxx>
---
drivers/acpi/system.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/acpi/system.c b/drivers/acpi/system.c
index 55cf4c0..4749f37 100644
--- a/drivers/acpi/system.c
+++ b/drivers/acpi/system.c
@@ -319,7 +319,7 @@ void acpi_irq_stats_init(void)
goto fail;
for (i = 0; i < num_counters; ++i) {
- char buffer[10];
+ char buffer[12];
char *name;
if (i < num_gpes)
--
1.5.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
