Skip to content

Commit

Permalink
target/sbp: Remove strict param from sbp_parse_wwn
Browse files Browse the repository at this point in the history
It's always set, and controls whether uppercase A-F are allowed hex values.
I don't see a reason not to accept these.

Signed-off-by: Andy Grover <agrover@redhat.com>
Cc: Chris Boot <bootc@bootc.net>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
  • Loading branch information
Andy Grover authored and Nicholas Bellinger committed Sep 17, 2012
1 parent 283669d commit 343d475
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions drivers/target/sbp/sbp_target.c
Original file line number Diff line number Diff line change
Expand Up @@ -2068,7 +2068,7 @@ static int sbp_update_unit_directory(struct sbp_tport *tport)
return ret;
}

static ssize_t sbp_parse_wwn(const char *name, u64 *wwn, int strict)
static ssize_t sbp_parse_wwn(const char *name, u64 *wwn)
{
const char *cp;
char c, nibble;
Expand All @@ -2088,7 +2088,7 @@ static ssize_t sbp_parse_wwn(const char *name, u64 *wwn, int strict)
err = 3;
if (isdigit(c))
nibble = c - '0';
else if (isxdigit(c) && (islower(c) || !strict))
else if (isxdigit(c))
nibble = tolower(c) - 'a' + 10;
else
goto fail;
Expand Down Expand Up @@ -2117,7 +2117,7 @@ static struct se_node_acl *sbp_make_nodeacl(
u64 guid = 0;
u32 nexus_depth = 1;

if (sbp_parse_wwn(name, &guid, 1) < 0)
if (sbp_parse_wwn(name, &guid) < 0)
return ERR_PTR(-EINVAL);

se_nacl_new = sbp_alloc_fabric_acl(se_tpg);
Expand Down Expand Up @@ -2253,7 +2253,7 @@ static struct se_wwn *sbp_make_tport(
struct sbp_tport *tport;
u64 guid = 0;

if (sbp_parse_wwn(name, &guid, 1) < 0)
if (sbp_parse_wwn(name, &guid) < 0)
return ERR_PTR(-EINVAL);

tport = kzalloc(sizeof(*tport), GFP_KERNEL);
Expand Down

0 comments on commit 343d475

Please sign in to comment.