Skip to contentSkip to conversation

apparmor: Strlen oops in 3.7.0-rc1 (mac80211-next)

PATCH Chaitanya 2 participants 3 messages Dec 13, 2012 – Dec 14, 2012

Subsystems (2)
lore
Share & tools
lore

Chaitanya

author
#0 lore raw

Hi Johannes/Al Viro,

Can you submit the below patch?

When moved to mac80211-next kernel was facing the crash related to strlen failure in the audit module. After some searching found a fix from Al Viro (http://lkml.indiana.edu/hypermail/linux/kernel/1210.2/01060.html) .

This is recurrent.

---
 security/apparmor/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
security/apparmor/Makefile +1 -1
index 7b3021c..5706b74 100644
@@ -57,7 +57,7 @@ cmd_make-rlim = echo "static const char *const rlim_names[RLIM_NLIMITS] = {" \
 
 $(obj)/capability.o : $(obj)/capability_names.h
 $(obj)/resource.o : $(obj)/rlim_names.h
-$(obj)/capability_names.h : $(srctree)/include/linux/capability.h \
+$(obj)/capability_names.h : $(srctree)/include/uapi/linux/capability.h \
                            $(src)/Makefile
        $(call cmd,make-caps)
 $(obj)/rlim_names.h : $(srctree)/include/uapi/asm-generic/resource.h \

Regards,
Chaitanya T K.

#1 lore raw

Hi Chaitanya,

Quoted 3 lines · Chaitanya

Hi Johannes/Al Viro,

Can you submit the below patch?

Why don't you submit it?

Read Documentation/SubmittingPatches for instructions on how to format your patch correctly.

Looking at what you've sent so far, you're pretty much there, all you need to do is:
1. Check that your email client is sending patches that are formatted correctly (send one to yourself and make sure it applies.)
2. Make the patch message a bit more descriptive, a listing of the error would be useful, as would some background on what introduced it
3. Sign it off
4. Send it to the correct maintainers of the appropriate parts of the kernel - check MAINTAINERS in the top level directory of the kernel tree - from the content of the patch, I'd guess that this is for the AppArmour maintainers

Once you've done all that, there shouldn't be any reason why it won't be applied, and once that happens, it'll make it's way to the mac80211-next tree shortly. You should CC Johannes and linux-wireless just to make sure that everyone's aware of it.

Thanks,

Julian Calaby

Quoted 28 lines

When moved to mac80211-next kernel was facing the crash related to strlen failure in the audit module. After some searching found a fix from Al Viro (http://lkml.indiana.edu/hypermail/linux/kernel/1210.2/01060.html) .

This is recurrent.

---
 security/apparmor/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
security/apparmor/Makefile +1 -1
index 7b3021c..5706b74 100644
@@ -57,7 +57,7 @@ cmd_make-rlim = echo "static const char *const rlim_names[RLIM_NLIMITS] = {" \
 
 $(obj)/capability.o : $(obj)/capability_names.h
 $(obj)/resource.o : $(obj)/rlim_names.h
-$(obj)/capability_names.h : $(srctree)/include/linux/capability.h \
+$(obj)/capability_names.h : $(srctree)/include/uapi/linux/capability.h \
                            $(src)/Makefile
        $(call cmd,make-caps)
 $(obj)/rlim_names.h : $(srctree)/include/uapi/asm-generic/resource.h \

Regards,
Chaitanya T K.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Signature · 5 lines
Julian Calaby

Email: [email protected]
Profile: http://www.google.com/profiles/julian.calaby/
.Plan: http://sites.google.com/site/juliancalaby/
author
#2 lore raw

Hi Julian,

I have no problems to submit the patch, but thought it would be good if the person who fixed it submits. If there is still no response from Viro, will submit the patch myself.

Thanks.

Quoted 62 lines · Julian Calaby

Hi Chaitanya,

Quoted 3 lines · Chaitanya

Hi Johannes/Al Viro,

Can you submit the below patch?

Why don't you submit it?

Read Documentation/SubmittingPatches for instructions on how to format your patch correctly.

Looking at what you've sent so far, you're pretty much there, all you need to do is:
1. Check that your email client is sending patches that are formatted correctly (send one to yourself and make sure it applies.)
2. Make the patch message a bit more descriptive, a listing of the error would be useful, as would some background on what introduced it
3. Sign it off
4. Send it to the correct maintainers of the appropriate parts of the kernel - check MAINTAINERS in the top level directory of the kernel tree - from the content of the patch, I'd guess that this is for the AppArmour maintainers

Once you've done all that, there shouldn't be any reason why it won't be applied, and once that happens, it'll make it's way to the mac80211-next tree shortly. You should CC Johannes and linux-wireless just to make sure that everyone's aware of it.

Thanks,

Julian Calaby

Quoted 28 lines

When moved to mac80211-next kernel was facing the crash related to strlen failure in the audit module. After some searching found a fix from Al Viro (http://lkml.indiana.edu/hypermail/linux/kernel/1210.2/01060.html) .

This is recurrent.

---
 security/apparmor/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
security/apparmor/Makefile +1 -1
index 7b3021c..5706b74 100644
@@ -57,7 +57,7 @@ cmd_make-rlim = echo "static const char *const rlim_names[RLIM_NLIMITS] = {" \
 
 $(obj)/capability.o : $(obj)/capability_names.h
 $(obj)/resource.o : $(obj)/rlim_names.h
-$(obj)/capability_names.h : $(srctree)/include/linux/capability.h \
+$(obj)/capability_names.h : $(srctree)/include/uapi/linux/capability.h \
                            $(src)/Makefile
        $(call cmd,make-caps)
 $(obj)/rlim_names.h : $(srctree)/include/uapi/asm-generic/resource.h \

Regards,
Chaitanya T K.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html