From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753607AbYE3XCy (ORCPT ); Fri, 30 May 2008 19:02:54 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753418AbYE3XCm (ORCPT ); Fri, 30 May 2008 19:02:42 -0400 Received: from mu-out-0910.google.com ([209.85.134.190]:32758 "EHLO mu-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752412AbYE3XCl (ORCPT ); Fri, 30 May 2008 19:02:41 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:to:cc:subject:message-id:references:mime-version:content-type:content-disposition:in-reply-to:user-agent:from; b=fiAK/V1MhS5l3hERf6rxw1T0wjOn7yBAomvSCA7BZWm4UAOvUYam7rn6a97DnUv+V2XLizlEi3e/Lw6mtkyMMQaqR83hPIAeWTfx59E1vvtrpy8ryB/mdvd6aoZguCtqOqOO62Yx1nK7XyF8EOnII2rsILYrlDMiYkNe+0AvfDs= Date: Sat, 31 May 2008 02:57:51 +0300 To: Casey Schaufler , Paul Moore Cc: linux-security-module@vger.kernel.org, LKML , netdev@vger.kernel.org, Andrew Morton Subject: [PATCH BUGFIX -v2 -rc4] Smack: Respect 'unlabeled' netlabel mode Message-ID: <20080530235751.GA6888@ubuntu> References: <20080530233603.GA2994@ubuntu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080530233603.GA2994@ubuntu> User-Agent: Mutt/1.5.15+20070412 (2007-04-11) From: "Ahmed S. Darwish" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi!, [ Sorry, Fix: Acquire smack_ambient_lock before reading smack_net_ambient ] --> In case of Smack 'unlabeled' netlabel option, Smack passes a _zero_ initialized 'secattr' to label a packet/sock. This causes an [unfound domain label error]/-ENOENT by netlbl_sock_setattr(). Above Netlabel failure leads to Smack socket hooks failure causing an always-on socket() -EPERM error. Such packets should have a netlabel domain agreed with netlabel to represent unlabeled packets. Fortunately Smack net ambient label packets are agreed with netlabel to be treated as unlabeled packets. Treat all packets coming out from a 'unlabeled' Smack system as coming from the smack net ambient label. Signed-off-by: Ahmed S. Darwish --- diff --git a/security/smack/smack.h b/security/smack/smack.h index 4a4477f..81b94ba 100644 --- a/security/smack/smack.h +++ b/security/smack/smack.h @@ -16,6 +16,7 @@ #include #include #include +#include #include /* @@ -178,6 +179,7 @@ u32 smack_to_secid(const char *); extern int smack_cipso_direct; extern int smack_net_nltype; extern char *smack_net_ambient; +extern struct mutex smack_ambient_lock; extern struct smack_known *smack_known; extern struct smack_known smack_known_floor; diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c index b5c8f92..0d9c7dc 100644 --- a/security/smack/smack_lsm.c +++ b/security/smack/smack_lsm.c @@ -1292,6 +1292,11 @@ static void smack_to_secattr(char *smack, struct netlbl_lsm_secattr *nlsp) } break; default: + mutex_lock(&smack_ambient_lock); + nlsp->domain = kstrdup(smack_net_ambient, GFP_ATOMIC); + mutex_unlock(&smack_ambient_lock); + + nlsp->flags = NETLBL_SECATTR_DOMAIN; break; } } diff --git a/security/smack/smackfs.c b/security/smack/smackfs.c index 271a835..cc357dc 100644 --- a/security/smack/smackfs.c +++ b/security/smack/smackfs.c @@ -46,7 +46,7 @@ enum smk_inos { */ static DEFINE_MUTEX(smack_list_lock); static DEFINE_MUTEX(smack_cipso_lock); -static DEFINE_MUTEX(smack_ambient_lock); +DEFINE_MUTEX(smack_ambient_lock); /* * This is the "ambient" label for network traffic. -- "Better to light a candle, than curse the darkness" Ahmed S. Darwish Homepage: http://darwish.07.googlepages.com Blog: http://darwish-07.blogspot.com