Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp735601ybi; Fri, 21 Jun 2019 07:18:57 -0700 (PDT) X-Google-Smtp-Source: APXvYqwQ7D3/muYTIaPAZAvNv/JnFlFQb14Zo2UaHMpWAtGg98KJchMBEpHNJExWPLrVuRJyUzhM X-Received: by 2002:a63:455c:: with SMTP id u28mr19264319pgk.416.1561126737469; Fri, 21 Jun 2019 07:18:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1561126737; cv=none; d=google.com; s=arc-20160816; b=iW/4+kJQQE+muBacWjgZP84yqsgw8NpjogGQj3rb05T6j/tA/cWk/ZzSeJsFQs4VOL nW8UKCSwNC1+c93zGow7JwlCu/CZjsUgevS2BWzkydv+yTJydBAXw4pH6mmbaJVfUTUl OGsRgZRA9YvVTq5zIJIJsHkJdG7qXqlIKQ0SZtD4XHU7uVwd2HK6zjTyHTlF11E8VBCL H00fsjQPi+nmkKgU1vGFdgGxZKoiUNz8GznbpcEZLaQBjQnaPZl/5+QiAN9LxIdEWgMy Q+XCNnLs6kD6eLjF33mDxDgekrHXMxTlDm96XCLirKixmCne/9U/+kqtmjOa+U/WEjCL mrWg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=gso8gaZxSpcxxBSqzAT7Fzf+sMSgRckufBmAJUDNrrM=; b=GdD+K9Z8YDprLwdRTTNO14GAmYGR2pmyCx6Emlbku6rmIzF1BBzcGUAukrs0ynqlwH ik9l3dM31QZ/xYVhn+d1a/roim+8vhH1YrgO+BSMkzTVGcrni++Smvqqo5eA42yEH6rJ hEXaSd0n1qL6B2R/s5zikWPX1f8qFKW0a1vPR/x6Qz4/gMJEpeCqb2+mTb7u6Ie2t/ak KXrYuhbq757ZcNvVo8Hc+iJeIZoIvtwcK45/n6hd9T45bwmwcNK2QMdTMP3O/Wj/lLQI 8uZ2zYHPMclwjQh/DbPtTi54+/RI0KTxhJBCvyESV/nzTegMPQgZA/wssmcXXTjCgUUP i9Mw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=eRaYWm9Y; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s6si2936427pjq.108.2019.06.21.07.18.41; Fri, 21 Jun 2019 07:18:57 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=eRaYWm9Y; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726134AbfFUOSg (ORCPT + 99 others); Fri, 21 Jun 2019 10:18:36 -0400 Received: from mail.kernel.org ([198.145.29.99]:48232 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725975AbfFUOSg (ORCPT ); Fri, 21 Jun 2019 10:18:36 -0400 Received: from tleilax.poochiereds.net (cpe-71-70-156-158.nc.res.rr.com [71.70.156.158]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id D0EBB20679; Fri, 21 Jun 2019 14:18:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1561126715; bh=utpNSfhfqDUGvBr9WSoyJ8v2U6SsAumLTAISUYtlQTY=; h=From:To:Cc:Subject:Date:From; b=eRaYWm9YQG+vjmKWshHLBOcgeJpq63PTM/hsvBnkJzVYfsqhUV51SAFEgR+v+xBPj IiM5mHFJnjB/ZeuUpREW6v6K9uiwDFWK3VR2+E661IoaaJmZQK8rpYIJbEBXKmIdG7 Bdz7S1xcCTEaaks8+NMyEWVgZ9aaeuLNZh1rA1OY= From: Jeff Layton To: linux-kernel@vger.kernel.org, ceph-devel@vger.kernel.org Cc: idryomov@gmail.com, zyan@redhat.com, sage@redhat.com, agruenba@redhat.com, joe@perches.com, geert+renesas@glider.be, andriy.shevchenko@linux.intel.com Subject: [PATCH v3 0/2] ceph: don't NULL terminate virtual xattr values Date: Fri, 21 Jun 2019 10:18:31 -0400 Message-Id: <20190621141833.17551-1-jlayton@kernel.org> X-Mailer: git-send-email 2.21.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org v3: switch to using an intermediate buffer for snprintf destination add patch to fix ceph_vxattrcb_layout return value v2: drop bogus EXPORT_SYMBOL of static function This is the 3rd posting of this patchset. Instead of adding a new snprintf variant that doesn't NULL terminate, this set instead has the vxattr handlers use an intermediate buffer as the snprintf destination and then memcpy's the result into the destination buffer. Also, I added a patch to fix up the return of ceph_vxattrcb_layout. The existing code actually worked, but relied on casting a signed negative value to unsigned and back, which seemed a little sketchy. Most of the rationale for this set is in the description of the first patch of the series. Jeff Layton (2): ceph: fix buffer length handling in virtual xattrs ceph: fix return of ceph_vxattrcb_layout fs/ceph/xattr.c | 113 ++++++++++++++++++++++++++++++++++-------------- 1 file changed, 81 insertions(+), 32 deletions(-) -- 2.21.0