From 351ee477f713730d1c53cf26b6fb87706d268a5f Mon Sep 17 00:00:00 2001 From: Chris Leech Date: Tue, 13 Aug 2013 12:39:07 -0700 Subject: [PATCH 1/1] idbw_rec_write, pick tpgt from existing record On a static add (-m node -o new) without a user specified tpgt, looks for existing new style records with tpgt before creating an old style record without. If one exists, take the tpgt from it an write an updated new style record instead. --- usr/idbm.c | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+) diff --git a/usr/idbm.c b/usr/idbm.c index b6193e7..2208c4a 100644 --- a/usr/idbm.c +++ b/usr/idbm.c @@ -28,6 +28,7 @@ #include #include #include +#include #include #include #include @@ -44,6 +45,10 @@ #include "fw_context.h" #include "iscsi_err.h" +#ifndef GLOB_ONLYDIR +#define GLOB_ONLYDIR 0x100 +#endif + #define IDBM_HIDE 0 /* Hide parameter when print. */ #define IDBM_SHOW 1 /* Show parameter when print. */ #define IDBM_MASKED 2 /* Show "stars" instead of real value when print */ @@ -202,6 +207,8 @@ static struct int_list_tbl { { "SHA3-256", AUTH_CHAP_ALG_SHA3_256 }, }; +static int idbm_remove_disc_to_node_link(node_rec_t *rec, char *portal); + static void idbm_recinfo_discovery(discovery_rec_t *r, recinfo_t *ri) { @@ -2206,12 +2213,49 @@ static int idbm_rec_write_old(node_rec_t *rec) FILE *f; char *portal; int rc = 0; + glob_t globbuf; + size_t i; + int tpgt = PORTAL_GROUP_TAG_UNKNOWN; portal = malloc(PATH_MAX); if (!portal) { log_error("Could not alloc portal"); return ISCSI_ERR_NOMEM; } + + /* check for newer portal dir with tpgt */ + snprintf(portal, PATH_MAX, "%s/%s/%s,%d,*", NODE_CONFIG_DIR, + rec->name, rec->conn[0].address, rec->conn[0].port); + rc = glob(portal, GLOB_ONLYDIR, NULL, &globbuf); + if (!rc) { + if (globbuf.gl_pathc > 1) + log_warning("multiple tpg records for portal " + "%s/%s:%d found", rec->name, + rec->conn[0].address, rec->conn[0].port); + /* set pattern for sscanf matching of tpgt */ + snprintf(portal, PATH_MAX, "%s/%s/%s,%d,%%u", NODE_CONFIG_DIR, + rec->name, rec->conn[0].address, rec->conn[0].port); + for (i = 0; i < globbuf.gl_pathc; i++) { + rc = sscanf(globbuf.gl_pathv[i], portal, &tpgt); + if (rc == 1) + break; + } + if (tpgt == PORTAL_GROUP_TAG_UNKNOWN) + log_warning("glob match on existing records, " + "but no valid tpgt found"); + } + globfree(&globbuf); + rc = 0; + + /* if a tpgt was selected from an old record, write entry in new format */ + if (tpgt != PORTAL_GROUP_TAG_UNKNOWN) { + log_warning("using tpgt %u from existing record", tpgt); + rec->tpgt = tpgt; + rc = idbm_remove_disc_to_node_link(rec, portal); + free(portal); + return idbm_rec_write_new(rec); + } + snprintf(portal, PATH_MAX, "%s/%s/%s,%d", NODE_CONFIG_DIR, rec->name, rec->conn[0].address, rec->conn[0].port);