[PATCH v2] fabrics: Handle space-padded TRSVCID and TRADDR fields

[J Freyensee]

On Mon, 2017-03-06 at 11:36 -0800, Roland Dreier wrote:
> From: Roland Dreier <roland at purestorage.com>
> 
> The TRSVCID and TRADDR fields in the discovery log page are defined
> as ASCII strings, which according to the NVMe standard means they
> should be space-padded rather than NUL-terminated.
> 
> The current nvme-cli code will print all the spaces and possibly some
> garbage from the next field.  For example this causes "connect-all"
> to write strings that get rejected with "malformed IP address passed."
> 
> Fix this by only writing the contents of these fields until the last
> non-space character, and limiting the length to the size of the field.
> 
> Signed-off-by: Roland Dreier <roland at purestorage.com>

Reviewed-by: Jay Freyensee <james_p_freyensee at linux.intel.com>

(already pulled into nvme-cli by Keith).