# HG changeset patch # User Richard Maciel <rmaciel(a)linux.vnet.ibm.com&gt; # Date 1243702984 10800 # Node ID 0b79dd93e4ede362b4da5bb83dba8c6bca0ab783 # Parent 1cb3975921d590d4dda4de197a4dc687e45d1840 Fixes bug #53409 To accomplish this, the lexer and the parser were changed to accept array of strings values for properties Signed-off-by: Richard Maciel <rmaciel(a)linux.vnet.ibm.com&gt; diff -r 1cb3975921d5 -r 0b79dd93e4ed eo_util_lexer.l --- a/eo_util_lexer.l Mon Apr 27 16:11:32 2009 -0700 +++ b/eo_util_lexer.l Sat May 30 14:03:04 2009 -0300 @@ -4,6 +4,7 @@ * Authors: * Gareth Bestor <bestorga(a)us.ibm.com&gt; * Dan Smith <danms(a)us.ibm.com&gt; + * Richard Maciel <richardm(a)br.ibm.com&gt; * */ /*** WARNING - COMMENTS IN LEX MUST BE TAB INDENTED ***/ @@ -91,7 +92,18 @@ return(STRING); } - /* Classname */ +\{ { + return(OPENBRACKET); + } + +\} { + return(CLOSEBRACKET); + } + +\, { + return(COMMA); + } + /* NOTE - this rule only applies after a 'INSTANCE OF' has been read in */ <READCLASSNAME>[A-Za-z][A-Za-z0-9_]* { BEGIN INITIAL; /* Go back to normal parsing rules now */ diff -r 1cb3975921d5 -r 0b79dd93e4ed eo_util_parser.y --- a/eo_util_parser.y Mon Apr 27 16:11:32 2009 -0700 +++ b/eo_util_parser.y Sat May 30 14:03:04 2009 -0300 @@ -4,6 +4,7 @@ * Authors: * Gareth Bestor <bestorga(a)us.ibm.com&gt; * Dan Smith <danms(a)us.ibm.com&gt; + * Richard Maciel <richardm(a)br.ibm.com&gt; */ /* DEFINITIONS SECTION */ @@ -20,15 +21,24 @@ /* specify prototypes to get rid of warnings */ int eo_parse_lex (void); void eo_parse_error(char *); +inline void ins_chars_into_cmstr_arr(const CMPIBroker *broker, + CMPIArray *arr, + CMPICount index, + char *str); #define RC_OK 0 #define RC_EOF EOF #define RC_INVALID_CLASS -1000 +#define EODEBUG 1 /* DEFINE ANY GLOBAL VARS HERE */ static const CMPIBroker * _BROKER; static CMPIInstance ** _INSTANCE; static const char * _NAMESPACE; +static CMPICount stringarraysize; +static char **stringarray; +static char *stringarraypropname; + #ifdef EODEBUG #define EOTRACE(fmt, arg...) fprintf(stderr, fmt, ##arg) @@ -36,6 +46,7 @@ #define EOTRACE(fmt, arg...) #endif + int eo_parse_parseinstance(const CMPIBroker *broker, CMPIInstance **instance, const char *ns); @@ -52,7 +63,7 @@ } /* Define simple (untyped) lexical tokens */ -%token INSTANCE OF ENDOFFILE +%token INSTANCE OF ENDOFFILE OPENBRACKET CLOSEBRACKET COMMA /* Define lexical tokens that return a value and their return type */ %token <string> CLASS @@ -67,7 +78,7 @@ /* Rules section */ instance: /* empty */ - | INSTANCE OF CLASS '{' + | INSTANCE OF CLASS OPENBRACKET { EOTRACE("classname = %s\n",$3); CMPIObjectPath *op; @@ -82,7 +93,7 @@ return RC_INVALID_CLASS; free($3); } - properties '}' ';' + properties CLOSEBRACKET ';' { /* Return after reading in each instance */ return RC_OK; @@ -127,7 +138,17 @@ CMSetProperty(*_INSTANCE, $1, &($3), CMPI_boolean); free($1); } + | PROPERTYNAME '=' OPENBRACKET + { + EOTRACE("propertyname = %s\n" + "\ttype = CMPI_charsA\n", + $1); + stringarraysize = 0; + stringarraypropname = $1; + } + arrayofstrings CLOSEBRACKET ';' + | PROPERTYNAME '=' CIMNULL ';' { EOTRACE("propertyname = %s\n" @@ -136,11 +157,86 @@ } ; +arrayofstrings: STRING + { + EOTRACE("BootDevices[%u]=%s\n",stringarraysize, $1); + + stringarraysize++; + stringarray = (char **)realloc(stringarray, + sizeof(char *) * + stringarraysize); + stringarray[stringarraysize-1] = $1; + } + COMMA arrayofstrings + + + | STRING + { + CMPIArray *arr; + CMPICount i; + CMPIStatus s; + + EOTRACE("\tBootDevices[%u]=%s\n",stringarraysize, $1); + + stringarraysize++; + + arr = CMNewArray(_BROKER, + stringarraysize, + CMPI_string, + &s); + if (s.rc != CMPI_RC_OK || CMIsNullObject(arr)) + EOTRACE("Error creating array\n"); + + // Values to stringarraysize - 2 are in the + // temporary array + for (i = 0; i < stringarraysize - 1; i++) { + ins_chars_into_cmstr_arr(_BROKER, + arr, + i, + stringarray[i]); + + free(stringarray[i]); + } + + ins_chars_into_cmstr_arr(_BROKER, + arr, + stringarraysize - 1, + $1); + + free($1); + + CMSetProperty(*_INSTANCE, + stringarraypropname, + &arr, + CMPI_stringA); + + free(stringarraypropname); + } + ; + /* END OF RULES SECTION */ %% /* USER SUBROUTINE SECTION */ +inline void ins_chars_into_cmstr_arr(const CMPIBroker *broker, + CMPIArray *arr, + CMPICount index, + char *str) +{ + CMPIString *cm_str; + CMPIStatus s; + + cm_str = CMNewString(_BROKER, str, &s); + if (s.rc != CMPI_RC_OK || CMIsNullObject(cm_str)) + EOTRACE("Error creating CMPIString"); + + s = CMSetArrayElementAt(arr, index, &cm_str, CMPI_string); + if (s.rc != CMPI_RC_OK) + EOTRACE("Error setting array element %u\n" + "Error code: %d\n", index, s.rc); +} + int eo_parse_parseinstance(const CMPIBroker *broker, CMPIInstance **instance, const char *ns)