From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas@monjalon.net>
Subject: Re: [dpdk-stable] [PATCH v3] examples/vhost_scsi: fix
	buffer not terminated
Date: Tue, 24 Oct 2017 18:22:43 +0200
Message-ID: <3010275.oDysHLUK20@xps>
References: <1507790695-10449-1-git-send-email-jacekx.piasecki@intel.com>
 <218441fc-8064-b05f-4eeb-4121dcdf849f@redhat.com>
 <20171017132637.GY1545@yliu-home>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: stable@dpdk.org, dev@dpdk.org, michalx.k.jastrzebski@intel.com,
 changpeng.liu@intel.com
To: Yuanhan Liu <yliu@fridaylinux.org>,
 Maxime Coquelin <maxime.coquelin@redhat.com>,
 Jacek Piasecki <jacekx.piasecki@intel.com>
Return-path: <dev-bounces@dpdk.org>
In-Reply-To: <20171017132637.GY1545@yliu-home>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

17/10/2017 15:26, Yuanhan Liu:
> On Fri, Oct 13, 2017 at 09:12:33AM +0200, Maxime Coquelin wrote:
> > On 10/12/2017 01:34 PM, Jacek Piasecki wrote:
> > >--- a/examples/vhost_scsi/scsi.c
> > >+++ b/examples/vhost_scsi/scsi.c
> > >@@ -307,7 +307,9 @@ vhost_bdev_scsi_inquiry_command(struct vhost_block_dev *bdev,
> > >  		strncpy((char *)inqdata->t10_vendor_id, "INTEL", 8);
> > >  		/* PRODUCT IDENTIFICATION */
> > >-		strncpy((char *)inqdata->product_id, bdev->product_name, 16);
> > >+		snprintf((char *)inqdata->product_id,
> > >+				ARRAY_SIZE(inqdata->product_id), "%s",
> > >+				bdev->product_name);
> > >  		/* PRODUCT REVISION LEVEL */
> > >  		strncpy((char *)inqdata->product_rev, "0001", 4);
> > 
> > Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> 
> Firstly, sorry for being so late response. And,
> 
> Applied to dpdk-next-virtio. Thanks!

I don't know where this ARRAY_SIZE comes from.
It does not compile.
In DPDK you can use RTE_DIM.

This patch is removed from the next-virtio pull queue.