| Message ID | 1418902316-30266-1-git-send-email-nhorman@tuxdriver.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 235CA7DF0; Thu, 18 Dec 2014 12:32:09 +0100 (CET) Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 33CC468BF for <dev@dpdk.org>; Thu, 18 Dec 2014 12:32:08 +0100 (CET) Received: from hmsreliant.think-freely.org ([2001:470:8:a08:7aac:c0ff:fec2:933b] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from <nhorman@tuxdriver.com>) id 1Y1ZJM-0003iO-64; Thu, 18 Dec 2014 06:32:06 -0500 From: Neil Horman <nhorman@tuxdriver.com> To: dev@dpdk.org Date: Thu, 18 Dec 2014 06:31:56 -0500 Message-Id: <1418902316-30266-1-git-send-email-nhorman@tuxdriver.com> X-Mailer: git-send-email 1.9.3 In-Reply-To: <1418835808-18803-1-git-send-email-nhorman@tuxdriver.com> References: <1418835808-18803-1-git-send-email-nhorman@tuxdriver.com> X-Spam-Score: -2.9 (--) X-Spam-Status: No Subject: [dpdk-dev] [PATCH v2] xenvirt: Fix build break on cmdline_parse_etheraddr call X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <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> |
Commit Message
Neil Horman
Dec. 18, 2014, 11:31 a.m. UTC
Back in:
commit aaa662e75c23c61a1d79bd4d1f9f35b4967c39db
Author: Alan Carew <alan.carew@intel.com>
Date: Fri Dec 5 15:19:07 2014 +0100
cmdline: fix overflow on bsd
The author failed to fixup a call to cmdline_parse_etheraddr in xenvirt. This
patch makes the needed correction to avoid a build break
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
CC: Thomas Monjalon <thomas.monjalon@6wind.com>
CC: Olivier Matz <olivier.matz@6wind.com>
---
Change notes
v2: Changed sizeof(struct ether_addr) to sizeof(dict->addr)
---
lib/librte_pmd_xenvirt/rte_eth_xenvirt.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
Comments
Hi Neil, I think if you could add the commit author in the cc list will be better. Because this could let him know about his code's issue and he is the always the best person to review the patch. Thanks, Michael On 2014/12/18 19:32, Neil Horman wrote: > Back in: > > commit aaa662e75c23c61a1d79bd4d1f9f35b4967c39db > Author: Alan Carew <alan.carew@intel.com> > Date: Fri Dec 5 15:19:07 2014 +0100 > > cmdline: fix overflow on bsd > > The author failed to fixup a call to cmdline_parse_etheraddr in xenvirt. This > patch makes the needed correction to avoid a build break > > Signed-off-by: Neil Horman <nhorman@tuxdriver.com> > CC: Thomas Monjalon <thomas.monjalon@6wind.com> > CC: Olivier Matz <olivier.matz@6wind.com> > > --- > Change notes > > v2: Changed sizeof(struct ether_addr) to sizeof(dict->addr) > --- > lib/librte_pmd_xenvirt/rte_eth_xenvirt.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c > index 6555ec5..04e30c9 100644 > --- a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c > +++ b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c > @@ -586,8 +586,9 @@ rte_eth_xenvirt_parse_args(struct xenvirt_dict *dict, > if (!strncmp(pair[0], RTE_ETH_XENVIRT_MAC_PARAM, > sizeof(RTE_ETH_XENVIRT_MAC_PARAM))) { > if (cmdline_parse_etheraddr(NULL, > - pair[1], > - &dict->addr) < 0) { > + pair[1], > + &dict->addr, > + sizeof(dict->addr)) < 0) { > RTE_LOG(ERR, PMD, > "Invalid %s device ether address\n", > name);
On Thu, Dec 18, 2014 at 01:45:54PM +0000, Qiu, Michael wrote: > Hi Neil, > > I think if you could add the commit author in the cc list will be better. > The commit author is me, and its recorded by the Signed-off line, as well as the Authorship tag in git. Not really sure what you're driving at here. > Because this could let him know about his code's issue and he is the > always the best person to review the patch. > Yes, Git already does that, as noted above. Neil > Thanks, > Michael > > On 2014/12/18 19:32, Neil Horman wrote: > > Back in: > > > > commit aaa662e75c23c61a1d79bd4d1f9f35b4967c39db > > Author: Alan Carew <alan.carew@intel.com> > > Date: Fri Dec 5 15:19:07 2014 +0100 > > > > cmdline: fix overflow on bsd > > > > The author failed to fixup a call to cmdline_parse_etheraddr in xenvirt. This > > patch makes the needed correction to avoid a build break > > > > Signed-off-by: Neil Horman <nhorman@tuxdriver.com> > > CC: Thomas Monjalon <thomas.monjalon@6wind.com> > > CC: Olivier Matz <olivier.matz@6wind.com> > > > > --- > > Change notes > > > > v2: Changed sizeof(struct ether_addr) to sizeof(dict->addr) > > --- > > lib/librte_pmd_xenvirt/rte_eth_xenvirt.c | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c > > index 6555ec5..04e30c9 100644 > > --- a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c > > +++ b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c > > @@ -586,8 +586,9 @@ rte_eth_xenvirt_parse_args(struct xenvirt_dict *dict, > > if (!strncmp(pair[0], RTE_ETH_XENVIRT_MAC_PARAM, > > sizeof(RTE_ETH_XENVIRT_MAC_PARAM))) { > > if (cmdline_parse_etheraddr(NULL, > > - pair[1], > > - &dict->addr) < 0) { > > + pair[1], > > + &dict->addr, > > + sizeof(dict->addr)) < 0) { > > RTE_LOG(ERR, PMD, > > "Invalid %s device ether address\n", > > name); > >
2014-12-18 06:31, Neil Horman: > Back in: > > commit aaa662e75c23c61a1d79bd4d1f9f35b4967c39db > Author: Alan Carew <alan.carew@intel.com> > Date: Fri Dec 5 15:19:07 2014 +0100 > > cmdline: fix overflow on bsd > > The author failed to fixup a call to cmdline_parse_etheraddr in xenvirt. This > patch makes the needed correction to avoid a build break > > Signed-off-by: Neil Horman <nhorman@tuxdriver.com> > CC: Thomas Monjalon <thomas.monjalon@6wind.com> > CC: Olivier Matz <olivier.matz@6wind.com> > > --- > Change notes > > v2: Changed sizeof(struct ether_addr) to sizeof(dict->addr) Applied Thanks
On 12/19/2014 4:54 AM, Neil Horman wrote: > On Thu, Dec 18, 2014 at 01:45:54PM +0000, Qiu, Michael wrote: >> Hi Neil, >> >> I think if you could add the commit author in the cc list will be better. >> > The commit author is me, and its recorded by the Signed-off line, as well as the > Authorship tag in git. Not really sure what you're driving at here. Sorry, What I mean "commit author" is who mentioned in you commit log, for this case should be this guy: alan.carew@intel.com I think. Sorry for I did not say it clearly. Yourself, of course, should be in the list. Thanks, Michael > >> Because this could let him know about his code's issue and he is the >> always the best person to review the patch. >> > Yes, Git already does that, as noted above. > > Neil > > >> Thanks, >> Michael >> >> On 2014/12/18 19:32, Neil Horman wrote: >>> Back in: >>> >>> commit aaa662e75c23c61a1d79bd4d1f9f35b4967c39db >>> Author: Alan Carew <alan.carew@intel.com> >>> Date: Fri Dec 5 15:19:07 2014 +0100 >>> >>> cmdline: fix overflow on bsd >>> >>> The author failed to fixup a call to cmdline_parse_etheraddr in xenvirt. This >>> patch makes the needed correction to avoid a build break >>> >>> Signed-off-by: Neil Horman <nhorman@tuxdriver.com> >>> CC: Thomas Monjalon <thomas.monjalon@6wind.com> >>> CC: Olivier Matz <olivier.matz@6wind.com> >>> >>> --- >>> Change notes >>> >>> v2: Changed sizeof(struct ether_addr) to sizeof(dict->addr) >>> --- >>> lib/librte_pmd_xenvirt/rte_eth_xenvirt.c | 5 +++-- >>> 1 file changed, 3 insertions(+), 2 deletions(-) >>> >>> diff --git a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c >>> index 6555ec5..04e30c9 100644 >>> --- a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c >>> +++ b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c >>> @@ -586,8 +586,9 @@ rte_eth_xenvirt_parse_args(struct xenvirt_dict *dict, >>> if (!strncmp(pair[0], RTE_ETH_XENVIRT_MAC_PARAM, >>> sizeof(RTE_ETH_XENVIRT_MAC_PARAM))) { >>> if (cmdline_parse_etheraddr(NULL, >>> - pair[1], >>> - &dict->addr) < 0) { >>> + pair[1], >>> + &dict->addr, >>> + sizeof(dict->addr)) < 0) { >>> RTE_LOG(ERR, PMD, >>> "Invalid %s device ether address\n", >>> name); >>
diff --git a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c index 6555ec5..04e30c9 100644 --- a/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c +++ b/lib/librte_pmd_xenvirt/rte_eth_xenvirt.c @@ -586,8 +586,9 @@ rte_eth_xenvirt_parse_args(struct xenvirt_dict *dict, if (!strncmp(pair[0], RTE_ETH_XENVIRT_MAC_PARAM, sizeof(RTE_ETH_XENVIRT_MAC_PARAM))) { if (cmdline_parse_etheraddr(NULL, - pair[1], - &dict->addr) < 0) { + pair[1], + &dict->addr, + sizeof(dict->addr)) < 0) { RTE_LOG(ERR, PMD, "Invalid %s device ether address\n", name);