Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753115AbaATKUs (ORCPT ); Mon, 20 Jan 2014 05:20:48 -0500 Received: from mail-pa0-f49.google.com ([209.85.220.49]:40758 "EHLO mail-pa0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751800AbaATKUq (ORCPT ); Mon, 20 Jan 2014 05:20:46 -0500 MIME-Version: 1.0 In-Reply-To: References: <1389953232-9428-1-git-send-email-piastry@etersoft.ru> Date: Mon, 20 Jan 2014 14:20:43 +0400 X-Google-Sender-Auth: 9ymmR4ttW4ZGP4DR_KcOyxBckaM Message-ID: Subject: Re: [PATCH v7 0/7] Add O_DENY* support for VFS and CIFS/NFS From: Pavel Shilovsky To: "Volker.Lendecke@SerNet.DE" Cc: Kernel Mailing List , linux-cifs , linux-fsdevel , Linux NFS Mailing list , wine-devel@winehq.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2014/1/20 Volker Lendecke : > Hi! > > On Fri, Jan 17, 2014 at 02:07:05PM +0400, Pavel Shilovsky wrote: >> If O_DENYDELETE flag is specified and the open succeded, >> any further unlink operation will fail with -ESHAREDENIED >> untill this open is closed. Now this flag is processed by >> VFS and CIFS filesystem. NFS returns -EINVAL for opens >> with this flag. > > This looks really, really good, thanks! > > One question: If Samba wants to open a file for delete > access, there's no corresponding flag in the open > permissions. There can be the case where Samba wants to open > *just* for future unlink, no read or write access required. > Is there a way to achieve this atomically correct? You can try to use O_PATH flag. It doesn't give you a delete access but should be ok because further deleting will be done without file descriptor -- through unlink syscall. -- Best regards, Pavel Shilovsky. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/