Bug 33732 (strlen) - Small strlen
Summary: Small strlen
Status: CLOSED CODE_FIX
Alias: strlen
Product: ACPI
Classification: Unclassified
Component: Other (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: acpi_other
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-19 20:27 UTC by Pavel Vasilyev
Modified: 2012-07-14 15:31 UTC (History)
3 users (show)

See Also:
Kernel Version: 2.6.39-rc4
Subsystem:
Regression: No
Bisected commit-id:

Attachments
Strlen fix (814 bytes, patch)
2011-04-19 20:27 UTC, Pavel Vasilyev 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Description Pavel Vasilyev 2011-04-19 20:27:36 UTC 
Created attachment 54732 [details]
Strlen fix

The word "enable" contains six characters, (strlen ("enable") -1), equal to five.
The word "disable" has seven characters, (strlen ("disable")-1), equal to six.

Perhaps the author confused with sizeof?!
Comment 1 Zhang Rui 2012-01-18 05:05:45 UTC 
thanks for the patch.
Len, can you apply this patch please?
Comment 2 Zhang Rui 2012-01-18 05:06:21 UTC 
BTW, Pavel,

can you resend the patch to linux-acpi@vger.kernel.org mailing list?
Comment 3 Len Brown 2012-06-05 04:02:24 UTC 
applied.
Comment 4 Florian Mickler 2012-07-01 09:49:00 UTC 
A patch referencing this bug report has been merged in Linux v3.5-rc5:

commit 9f132652d94c96476b0b0a8caf0c10e96ab10fa8
Author: Pavel Vasilyev <pavel@pavlinux.ru>
Date:   Tue Jun 5 00:02:05 2012 -0400

    ACPI sysfs.c strlen fix
Note You need to log in before you can comment on or make changes to this bug.