Menu

#2150 Error in numeric keys

closed-fixed
5
2011-03-15
2011-02-28
Todd Bates
No

Numeric keys are encoded as doubles. Considering doubles are not allowed as keys this seems to be an error. For example if a unit32 key value encoded as 0.0 rather than 0.

Discussion

  • Dave Blaschke

    Dave Blaschke - 2011-02-28

    Could you please post the XML request/response that pointed this issue out

     
  • Dave Blaschke

    Dave Blaschke - 2011-02-28
    • assigned_to: nobody --> blaschke-oss
     
  • Dave Blaschke

    Dave Blaschke - 2011-02-28

    Oops, misread the "encoded" part... any chance you could post a Java snippet of how you're generating the request? or a toString() of the instance?

     
  • Todd Bates

    Todd Bates - 2011-02-28
     
  • Todd Bates

    Todd Bates - 2011-02-28
     
  • Todd Bates

    Todd Bates - 2011-02-28

    The cop is declared with

    protected static final CIMObjectPath Demo_MethodClass_INSTANCCE = new CIMObjectPath(
    "root/demo:Demo_MethodClass.Id=0")

    the actual call is

    final CIMArgument<?>[] inValues = new CIMArgument[0];
    final CIMArgument<?>[] outValues = new CIMArgument[20];
    final Object value = client.invokeMethod(Demo_MethodClass_INSTANCCE,
    "No_Args", inValues, outValues);

     
  • Dave Blaschke

    Dave Blaschke - 2011-03-01

    Seems like you stumbled on a special case with Id=0 because Id=-1, Id=1, etc. all seem to work fine. Heck, even Id=00 works, as does Id=0x0, just not Id=0.

     
  • Dave Blaschke

    Dave Blaschke - 2011-03-01
    • status: open --> open-fixed
     
  • Dave Blaschke

    Dave Blaschke - 2011-03-01

    Patch sent for community review. During a 2 week period any
    exploiter may comment on the patch, request changes or turn it
    down completely (with good reason). For the time being the patch is part of the "Experimental" branch in CVS.

     
  • Dave Blaschke

    Dave Blaschke - 2011-03-14

    Patch against HEAD

     
  • Dave Blaschke

    Dave Blaschke - 2011-03-14

    The community review has completed and we received no substantial critisism. Therefore the patch has been approved and merged into the "HEAD" branch. The next release will pick it up.

     
  • Dave Blaschke

    Dave Blaschke - 2011-03-14
    • status: open-fixed --> pending-fixed
     
  • Dave Blaschke

    Dave Blaschke - 2011-03-15

    The patch was picked up by release 2.1.8 and will therefore be closed.

     
  • Dave Blaschke

    Dave Blaschke - 2011-03-15
    • status: pending-fixed --> closed-fixed
     

Log in to post a comment.