> -----Original Message-----
> From: dev <[email protected]> On Behalf Of Gaetan Rivet
> Sent: Wednesday, May 13, 2020 12:48 PM
> To: [email protected]
> Subject: [dpdk-dev] [PATCH v1 2/2] pci: explain how empty strings are rejected
> in DBDF
> 
> Empty strings are forbidden as input to rte_pci_addr_parse().
> It is explicitly enforced in BDF parsing as parsing the bus
> field will immediately fail. The related check is commented.
> 
> It is implicitly enforced in DBDF parsing, as the domain would be
> parsed to 0 without error, but the check `end[0] != ':'` afterward
> will return -EINVAL.
> 
> Enforcing consistency between parsers by reading the code is not helped
> by this property being implicit. Add a comment to explain.
> 
> Signed-off-by: Gaetan Rivet <[email protected]>
> ---

Acked-by: Darek Stojaczyk <[email protected]>

Reply via email to