From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D788FC433F5 for ; Mon, 27 Sep 2021 21:40:11 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1049161156 for ; Mon, 27 Sep 2021 21:40:11 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 1049161156 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=nongnu.org Received: from localhost ([::1]:50032 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mUyM5-0006VE-Sh for qemu-devel@archiver.kernel.org; Mon, 27 Sep 2021 17:40:09 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51890) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mUyLJ-0005eF-BH for qemu-devel@nongnu.org; Mon, 27 Sep 2021 17:39:22 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:41212) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mUyLE-0007qe-MU for qemu-devel@nongnu.org; Mon, 27 Sep 2021 17:39:19 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1632778754; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=xAbeqsqHl7XKu+qyFe4uCm+8luAIN9qM4EDXm8i4TVU=; b=KCk8lKgdZpoO4ba93IE6dA345cDu+Wy//neTrYYNPAauy5fhxHk1wuphxwMOgo4HwZQMdq xYT7rabV0IVKwTEXKN2gTTRMo6n9BKYQeZcvK7MHnFypAlHYXS4HHrdUmM9TbucAm0tPef cYI++MrxifW1I4Q1CdYUZb7z5deXgd0= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-282-sHtDOM_mPiayhlmZCJMgNw-1; Mon, 27 Sep 2021 17:39:13 -0400 X-MC-Unique: sHtDOM_mPiayhlmZCJMgNw-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 93C25802B9F; Mon, 27 Sep 2021 21:39:12 +0000 (UTC) Received: from localhost (unknown [10.39.194.55]) by smtp.corp.redhat.com (Postfix) with ESMTP id A8A131F459; Mon, 27 Sep 2021 21:39:07 +0000 (UTC) Date: Mon, 27 Sep 2021 22:39:06 +0100 From: "Richard W.M. Jones" To: Eric Blake Subject: Re: [PATCH v2] nbd/server: Add --selinux-label option Message-ID: <20210927213906.GW3361@redhat.com> References: <20210723103303.1731437-1-rjones@redhat.com> <20210723103303.1731437-2-rjones@redhat.com> <20210927211834.tzqpx4egzwbvjmrs@redhat.com> MIME-Version: 1.0 In-Reply-To: <20210927211834.tzqpx4egzwbvjmrs@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=rjones@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Received-SPF: pass client-ip=170.10.133.124; envelope-from=rjones@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: vsementsov@virtuozzo.com, berrange@redhat.com, qemu-devel@nongnu.org, qemu-block@nongnu.org Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On Mon, Sep 27, 2021 at 04:18:34PM -0500, Eric Blake wrote: > On Fri, Jul 23, 2021 at 11:33:03AM +0100, Richard W.M. Jones wrote: > > Under SELinux, Unix domain sockets have two labels. One is on the > > disk and can be set with commands such as chcon(1). There is a > > different label stored in memory (called the process label). This can > > only be set by the process creating the socket. When using SELinux + > > SVirt and wanting qemu to be able to connect to a qemu-nbd instance, > > you must set both labels correctly first. > > > > For qemu-nbd the options to set the second label are awkward. You can > > create the socket in a wrapper program and then exec into qemu-nbd. > > Or you could try something with LD_PRELOAD. > > > > This commit adds the ability to set the label straightforwardly on the > > command line, via the new --selinux-label flag. (The name of the flag > > is the same as the equivalent nbdkit option.) > > > > A worked example showing how to use the new option can be found in > > this bug: https://bugzilla.redhat.com/show_bug.cgi?id=1984938 > > > > Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1984938 > > Signed-off-by: Richard W.M. Jones > > --- > > I'm making one tweak to your patch before sending the pull request: > > > +++ b/qemu-nbd.c > > @@ -64,6 +68,7 @@ > > #define QEMU_NBD_OPT_FORK 263 > > #define QEMU_NBD_OPT_TLSAUTHZ 264 > > #define QEMU_NBD_OPT_PID_FILE 265 > > +#define QEMU_NBD_OPT_SELINUX_LABEL 266 > > > > #define MBR_SIZE 512 > > > > @@ -116,6 +121,9 @@ static void usage(const char *name) > > " --fork fork off the server process and exit the parent\n" > > " once the server is running\n" > > " --pid-file=PATH store the server's process ID in the given file\n" > > +#ifdef CONFIG_SELINUX > > +" --selinux-label=LABEL set SELinux process label on listening socket\n" > > +#endif > > The new option is only conditionally advertised under --help (qemu-nbd > lacks a stable machine-parseable output, so scraping --help output > will have to do for now)... > > > #if HAVE_NBD_DEVICE > > "\n" > > "Kernel NBD client support:\n" > > @@ -532,6 +540,8 @@ int main(int argc, char **argv) > > { "trace", required_argument, NULL, 'T' }, > > { "fork", no_argument, NULL, QEMU_NBD_OPT_FORK }, > > { "pid-file", required_argument, NULL, QEMU_NBD_OPT_PID_FILE }, > > + { "selinux-label", required_argument, NULL, > > + QEMU_NBD_OPT_SELINUX_LABEL }, > > ...but is unconditionally supported as a long option even when support > was not compiled in... > > > { NULL, 0, NULL, 0 } > > }; > > int ch; > > @@ -558,6 +568,7 @@ int main(int argc, char **argv) > > int old_stderr = -1; > > unsigned socket_activation; > > const char *pid_file_name = NULL; > > + const char *selinux_label = NULL; > > BlockExportOptions *export_opts; > > > > #ifdef CONFIG_POSIX > > @@ -747,6 +758,9 @@ int main(int argc, char **argv) > > case QEMU_NBD_OPT_PID_FILE: > > pid_file_name = optarg; > > break; > > + case QEMU_NBD_OPT_SELINUX_LABEL: > > + selinux_label = optarg; > > + break; > > } > > } > > > > @@ -938,6 +952,16 @@ int main(int argc, char **argv) > > } else { > > backlog = MIN(shared, SOMAXCONN); > > } > > + if (sockpath && selinux_label) { > > +#ifdef CONFIG_SELINUX > > + if (setsockcreatecon_raw(selinux_label) == -1) { > > + error_report("Cannot set SELinux socket create context " > > + "to %s: %s", > > + selinux_label, strerror(errno)); > > + exit(EXIT_FAILURE); > > + } > > +#endif > > ...but here we silently ignore it if support is not compiled in. > Better is to issue an error message about using an unsupported option, > so I'll squash this in: > > diff --git i/qemu-nbd.c w/qemu-nbd.c > index 5dc82c419255..94f8ec07c064 100644 > --- i/qemu-nbd.c > +++ w/qemu-nbd.c > @@ -962,6 +962,9 @@ int main(int argc, char **argv) > selinux_label, strerror(errno)); > exit(EXIT_FAILURE); > } > +#else > + error_report("SELinux support not enabled in this binary"); > + exit(EXIT_FAILURE); > #endif > } > saddr = nbd_build_socket_address(sockpath, bindto, port); > @@ -978,6 +981,9 @@ int main(int argc, char **argv) > strerror(errno)); > exit(EXIT_FAILURE); > } > +#else > + error_report("SELinux support not enabled in this binary"); > + exit(EXIT_FAILURE); > #endif > } > } else { > Good idea, thanks. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-df lists disk usage of guests without needing to install any software inside the virtual machine. Supports Linux and Windows. http://people.redhat.com/~rjones/virt-df/