osaf/tools/safimm/immcfg/imm_cfg.c     |   31 ++++-
 osaf/tools/safimm/immcfg/imm_import.cc |  172 ++++++++++++++++++++++++++++++++-
 2 files changed, 193 insertions(+), 10 deletions(-)


For forwards compatibility, OpenSAF schema, where unknown attribute flags are 
defined, must be provided to immcfg using -X flag.

diff --git a/osaf/tools/safimm/immcfg/imm_cfg.c 
b/osaf/tools/safimm/immcfg/imm_cfg.c
--- a/osaf/tools/safimm/immcfg/imm_cfg.c
+++ b/osaf/tools/safimm/immcfg/imm_cfg.c
@@ -65,7 +65,7 @@ typedef enum {
 #define VERBOSE_INFO(format, args...) if (verbose) { fprintf(stderr, format, 
##args); }
 
 // Interface functions which implement -f and -L options (imm_import.cc)
-int importImmXML(char* xmlfileC, char* adminOwnerName, int verbose, int 
ccb_safe);
+int importImmXML(char* xmlfileC, char* adminOwnerName, int verbose, int 
ccb_safe, const char *xsdPath);
 int validateImmXML(const char *xmlfile, int verbose);
 
 
@@ -103,6 +103,7 @@ static void usage(const char *progname)
        printf("\t-L, --validate <imm.xml file>\n");
        printf("\t-o, --admin-owner <admin owner name>\n");
        printf("\t--admin-owner-clear\n");
+       printf("\t-X, --xsd <path_to_schema.xsd>\n");
 
        printf("\nEXAMPLE\n");
        printf("\timmcfg -a saAmfNodeSuFailoverMax=7 
safAmfNode=Node01,safAmfCluster=1\n");
@@ -127,6 +128,10 @@ static void usage(const char *progname)
        printf("\t\tuse 'myAdminOwnerName' as admin owner name for changing one 
attribute of one object\n");
        printf("\timmcfg --admin-owner-clear 
safAmfNode=Node01,safAmfCluster=1\n");
        printf("\t\tclear admin owner from one object\n");
+       printf("\timmcfg -X /etc/opensaf/schema.xsd -f imm.xml\n");
+       printf("\t\timmcfg will load unsupported attribute flags in the current 
OpenSAF version from /etc/opensaf/schema.xsd, and use them to successfully 
import imm.xml");
+       printf("\timmcfg -X /etc/opensaf -f imm.xml\n");
+       printf("\t\timmcfg will load unsupported attribute flags in the current 
OpenSAF version from the schema specified in imm.xml which is stored in 
/etc/opensaf, and use loaded flags to successfully import imm.xml");
 }
 
 /* signal handler for SIGALRM */
@@ -854,6 +859,7 @@ int main(int argc, char *argv[])
                {"disable-attr-notify", no_argument, NULL, 0},
                {"admin-owner", required_argument, NULL, 'o'},
                {"admin-owner-clear", no_argument, NULL, 0},
+               {"xsd", required_argument, NULL, 'X'},
                {0, 0, 0, 0}
        };
        SaAisErrorT error;
@@ -879,9 +885,11 @@ int main(int argc, char *argv[])
        unsigned long timeoutVal = 60;
        attr_notify_t attrNotify = NOTIFY_UNDEFINED;
 
+       char *xsdPath = NULL;
+
        while (1) {
                int option_index = 0;
-               c = getopt_long(argc, argv, "a:c:f:t:dhmvuL:o:", long_options, 
&option_index);
+               c = getopt_long(argc, argv, "a:c:f:t:dhmvuL:o:X:", 
long_options, &option_index);
 
                if (c == -1)    /* have all command-line options have been 
parsed? */
                        break;
@@ -963,6 +971,13 @@ int main(int argc, char *argv[])
                        adminOwnerName = 
(SaImmAdminOwnerNameT)malloc(strlen(optarg) + 1);
                        strcpy(adminOwnerName, optarg);
                        break;
+               case 'X':
+                       if(xsdPath) {
+                               fprintf(stderr, "XSD path is already set\n");
+                               exit(EXIT_FAILURE);
+                       }
+                       xsdPath = strdup(optarg);
+                       break;
                default:
                        free(adminOwnerName);
                        fprintf(stderr, "Try '%s --help' for more 
information\n", argv[0]);
@@ -994,7 +1009,7 @@ int main(int argc, char *argv[])
        
        if (op == LOAD_IMMFILE) {
                VERBOSE_INFO("importImmXML(xmlFilename=%s, verbose=%d)\n", 
xmlFilename, verbose);
-               rc = importImmXML(xmlFilename, adminOwnerName, verbose, 
ccb_safe);
+               rc = importImmXML(xmlFilename, adminOwnerName, verbose, 
ccb_safe, xsdPath);
                free(adminOwnerName);
                exit(rc);
        }
@@ -1113,10 +1128,14 @@ int main(int argc, char *argv[])
                free(attributeNames);
        error = immutil_saImmOmFinalize(immHandle);
        if (SA_AIS_OK != error) {
-                fprintf(stderr, "error - saImmOmFinalize FAILED: %s\n", 
saf_error(error));
-                rc = EXIT_FAILURE;
-        }
+               fprintf(stderr, "error - saImmOmFinalize FAILED: %s\n", 
saf_error(error));
+               rc = EXIT_FAILURE;
+       }
 
+       if(xsdPath) {
+               free(xsdPath);
+               xsdPath = NULL;
+       }
 
        exit(rc);
 }
diff --git a/osaf/tools/safimm/immcfg/imm_import.cc 
b/osaf/tools/safimm/immcfg/imm_import.cc
--- a/osaf/tools/safimm/immcfg/imm_import.cc
+++ b/osaf/tools/safimm/immcfg/imm_import.cc
@@ -21,6 +21,8 @@
 #include <set>
 #include <string>
 #include <libxml/parser.h>
+#include <libxml/tree.h>
+#include <libxml/xpath.h>
 #include <string.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -28,6 +30,8 @@
 #include <assert.h>
 #include <syslog.h>
 #include <configmake.h>
+#include <sys/stat.h>
+#include <errno.h>
 
 #include <saAis.h>
 #include <saImmOm.h>
@@ -58,7 +62,7 @@ static char base64_dec_table[] = {
 
 extern "C"
 {
-       int importImmXML(char* xmlfileC, char* adminOwnerName, int verbose, int 
ccb_safe);
+       int importImmXML(char* xmlfileC, char* adminOwnerName, int verbose, int 
ccb_safe, const char *xsdPath);
        int validateImmXML(const char *xmlfile, int verbose);
 }
 
@@ -134,6 +138,10 @@ typedef struct ParserStateStruct {
        bool validation;
 } ParserState;
 
+bool isXsdLoaded = false;
+static const char *imm_xsd_file;
+typedef std::set<std::string> AttrFlagSet;
+AttrFlagSet attrFlagSet;
 
 /* Prototypes */
 
@@ -1039,6 +1047,144 @@ static inline bool isBase64Encoded(const
        return isB64;
 }
 
+static inline char *getAttrValue(xmlAttributePtr attr) {
+    if(!attr || !attr->children) {
+        return NULL;
+    }
+
+    return (char *)attr->children->content;
+}
+
+static bool loadXsd(const xmlChar** attrs) {
+    if(!imm_xsd_file) {
+        return true;
+    }
+
+    // Check if schema path exist
+    struct stat st;
+    if(stat(imm_xsd_file, &st)) {
+        if(errno == ENOENT) {
+            LOG_ER("%s does not exist", imm_xsd_file);
+        } else {
+            LOG_ER("stat of %s return error: %d", imm_xsd_file, errno);
+        }
+
+        return false;
+    }
+    // It should be a file or a directory
+    if(!S_ISREG(st.st_mode) && !S_ISDIR(st.st_mode)) {
+        LOG_ER("%s is not a file or directory", imm_xsd_file);
+        return false;
+    }
+
+    std::string xsdFile = imm_xsd_file;
+    if(S_ISDIR(st.st_mode)) {
+        // Schema path is a directory, so we shall add a schema file from XML 
file
+        if(xsdFile.at(xsdFile.size() - 1) != '/') {
+            xsdFile.append("/");
+        }
+        char *xsd = (char *)getAttributeValue(attrs, (xmlChar 
*)"noNamespaceSchemaLocation");
+        if(!xsd) {
+            // try with a namespace
+            xsd = (char *)getAttributeValue(attrs, (xmlChar 
*)"xsi:noNamespaceSchemaLocation");
+            if(!xsd) {
+                LOG_ER("Schema is not defined in XML file");
+                return false;
+            }
+        }
+        xsdFile.append(xsd);
+
+        // Check if schema file exists and that it's a file
+        if(stat(xsdFile.c_str(), &st)) {
+            if(errno == ENOENT) {
+                LOG_ER("XSD file %s does not exist", imm_xsd_file);
+            } else {
+                LOG_ER("Stat of XSD file %s return error: %d", imm_xsd_file, 
errno);
+            }
+
+            return false;
+        }
+        if(!S_ISREG(st.st_mode)) {
+            LOG_ER("Schema %s is not a file", xsdFile.c_str());
+            return false;
+        }
+    }
+
+    xmlNodePtr xsdDocRoot;
+    xmlDocPtr xsdDoc = xmlParseFile(xsdFile.c_str());
+    if(!xsdDoc) {
+        return false;
+    }
+
+    bool rc = true;
+    xmlXPathContextPtr ctx = xmlXPathNewContext(xsdDoc);
+    if(!ctx) {
+        rc = false;
+        goto freedoc;
+    }
+
+    // Add namespace of the first element
+    xsdDocRoot = xmlDocGetRootElement(xsdDoc);
+    if(xsdDocRoot->ns) {
+        ctx->namespaces = (xmlNsPtr *)malloc(sizeof(xmlNsPtr));
+        ctx->namespaces[0] = xsdDocRoot->ns;
+        ctx->nsNr = 1;
+    }
+
+    xmlXPathObjectPtr xpathObj;
+    xpathObj = xmlXPathEval((const 
xmlChar*)"/xs:schema/xs:simpleType[@name=\"attr-flags\"]/xs:restriction/xs:enumeration",
 ctx);
+    if(!xpathObj || !xpathObj->nodesetval) {
+        rc = false;
+        goto freectx;
+    }
+
+    xmlElementPtr element;
+    xmlAttributePtr attr;
+    char *value;
+    int size;
+
+    size = xpathObj->nodesetval->nodeNr;
+    for(int i=0; i<size; i++) {
+        value = NULL;
+        element = (xmlElementPtr)xpathObj->nodesetval->nodeTab[i];
+        attr = element->attributes;
+        while(attr) {
+            if(!strcmp((char *)attr->name, "value")) {
+                value = getAttrValue(attr);
+            }
+
+            if(value) {
+                break;
+            }
+
+            attr = (xmlAttributePtr)attr->next;
+        }
+
+        if(value) {
+            if(strcmp(value, "SA_RUNTIME") && strcmp(value, "SA_CONFIG") &&
+                    strcmp(value, "SA_MULTI_VALUE") && strcmp(value, 
"SA_WRITABLE") &&
+                    strcmp(value, "SA_INITIALIZED") && strcmp(value, 
"SA_PERSISTENT") &&
+                    strcmp(value, "SA_CACHED") && strcmp(value, "SA_NOTIFY") &&
+                    strcmp(value, "SA_NO_DUPLICATES")) {
+                attrFlagSet.insert(value);
+            }
+        }
+    }
+
+    isXsdLoaded = true;
+
+    xmlXPathFreeObject(xpathObj);
+freectx:
+    if(ctx->nsNr) {
+        free(ctx->namespaces);
+    }
+    xmlXPathFreeContext(ctx);
+freedoc:
+    xmlFreeDoc(xsdDoc);
+
+    return rc;
+}
+
 /**
  * This is the handler for start tags
  */
@@ -1163,6 +1309,12 @@ static void startElementHandler(void* us
                /* <imm:IMM-contents> */
        } else if (strcmp((const char*)name, "imm:IMM-contents") == 0) {
                state->state[state->depth] = IMM_CONTENTS;
+               if(imm_xsd_file) {
+                       if(!loadXsd(attrs)) {
+                               LOG_ER("Failed to load XML schema", name);
+                               exit(1);
+                       }
+               }
        } else {
                LOG_ER("UNKNOWN TAG! (%s)", name);
                exit(1);
@@ -1614,9 +1766,17 @@ static SaImmAttrFlagsT charsToFlagsHelpe
                return SA_IMM_ATTR_NO_DUPLICATES;
        }
 
-       LOG_ER("UNKNOWN FLAGS, %s", str);
+    std::string flag((char *)str, len);
+    if(isXsdLoaded) {
+        AttrFlagSet::iterator it = attrFlagSet.find(flag);
+        if(it != attrFlagSet.end()) {
+            return 0;
+        }
+    }
 
-       exit(1);
+    LOG_ER("UNKNOWN FLAGS, %s", flag.c_str());
+
+    exit(1);
 }
 
 /**
@@ -2110,7 +2270,7 @@ int loadImmXML(std::string xmlfile)
 // C and c++ caller wrapper
 //  The objective is to keep the code copied from imm_load.cc as close to 
original as possible
 //  to ease a future refactoring towards common codebase
-int importImmXML(char* xmlfileC, char* adminOwnerName, int verbose, int 
ccb_safe)
+int importImmXML(char* xmlfileC, char* adminOwnerName, int verbose, int 
ccb_safe, const char *xsdPath)
 {
        std::string xmlfile(xmlfileC);
        imm_import_adminOwnerName = adminOwnerName;
@@ -2118,6 +2278,10 @@ int importImmXML(char* xmlfileC, char* a
        imm_import_ccb_safe = ccb_safe;
        LOG_IN("file: %s adminOwner: %s", xmlfileC, adminOwnerName);
 
+       imm_xsd_file = xsdPath;
+       isXsdLoaded = false;
+       attrFlagSet.clear();
+
        // assign own immutil errorhandler (no call to abort())
        immutilError = imm_importImmutilError;
 

------------------------------------------------------------------------------
Managing the Performance of Cloud-Based Applications
Take advantage of what the Cloud has to offer - Avoid Common Pitfalls.
Read the Whitepaper.
http://pubads.g.doubleclick.net/gampad/clk?id=121054471&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to