[opensuse-kernel] 2.6.22.7-13.1 warning
nux-2.6.22.7-13.1/security/apparmor/lsm.c:948: warning: initialization from inco static int apparmor_file_mmap(struct file *file, unsigned long reqprot, unsigned long prot, unsigned long flags, unsigned long addr, unsigned long addr_only) { return aa_mmap(file, "file_mmap", prot, flags); } The file_mmap hook in struct security_operations does not have addr_only. This is already the patched SUSE variant. -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-kernel+help@opensuse.org
On Tue, Sep 25, 2007 at 06:54:04PM +0200, Jan Engelhardt wrote:
nux-2.6.22.7-13.1/security/apparmor/lsm.c:948: warning: initialization from inco
static int apparmor_file_mmap(struct file *file, unsigned long reqprot, unsigned long prot, unsigned long flags, unsigned long addr, unsigned long addr_only) { return aa_mmap(file, "file_mmap", prot, flags); }
The file_mmap hook in struct security_operations does not have addr_only. This is already the patched SUSE variant.
I don't understand. Is the current SuSE kernel incorrectly building for you? Or is some other patch causing this problem? thanks, greg k-h -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-kernel+help@opensuse.org
On Tuesday 25 September 2007 20:04, Greg KH wrote:
On Tue, Sep 25, 2007 at 06:54:04PM +0200, Jan Engelhardt wrote:
nux-2.6.22.7-13.1/security/apparmor/lsm.c:948: warning: initialization from inco
static int apparmor_file_mmap(struct file *file, unsigned long reqprot, unsigned long prot, unsigned long flags, unsigned long addr, unsigned long addr_only) { return aa_mmap(file, "file_mmap", prot, flags); }
The file_mmap hook in struct security_operations does not have addr_only. This is already the patched SUSE variant.
I don't understand. Is the current SuSE kernel incorrectly building for you? Or is some other patch causing this problem?
The file_mmap hook and AppArmor's implementation simply don't match. The AA team is working on a fix. https://bugzilla.novell.com/328423 Thanks, Andreas -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-kernel+help@opensuse.org
On Sep 25 2007 11:04, Greg KH wrote:
On Tue, Sep 25, 2007 at 06:54:04PM +0200, Jan Engelhardt wrote:
nux-2.6.22.7-13.1/security/apparmor/lsm.c:948: warning: initialization from inco
static int apparmor_file_mmap(struct file *file, unsigned long reqprot, unsigned long prot, unsigned long flags, unsigned long addr, unsigned long addr_only) { return aa_mmap(file, "file_mmap", prot, flags); }
The file_mmap hook in struct security_operations does not have addr_only. This is already the patched SUSE variant.
I don't understand. Is the current SuSE kernel incorrectly building for you? Or is some other patch causing this problem?
In theory: If by "incorrectly building" you mean "spews warning messages that should have been paid attention to", then yes. In practice: It does not harm, since x86 cdecl calling convention is used and addr_only is not used. -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-kernel+help@opensuse.org
On Wednesday 26 September 2007 10:13, Jan Engelhardt wrote:
In practice: It does not harm, since x86 cdecl calling convention is used and addr_only is not used.
flags and prot end up with the wrong value though :( Andreas -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-kernel+help@opensuse.org
participants (3)
-
Andreas Gruenbacher -
Greg KH -
Jan Engelhardt