[Sblim-issues] [ sblim-Bugs-2021888 ] Bug in creating CIM instance

[SourceForge.net <no...@so...>]

Bugs item #2021888, was opened at 2008-07-18 18:29
Message generated for change (Comment added) made by mchasal
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=712784&aid=2021888&group_id=128809

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: sfcb
Group: Function
>Status: Pending
>Resolution: Fixed
Priority: 6
Private: No
Submitted By: Luong Nguyen-Duy (luongnguyenduy)
Assigned to: Michael Chase-Salerno (mchasal)
Summary: Bug in creating CIM instance

Initial Comment:

I am using sfcb 1.3.1

I found a bug in instance.c::internal_new_CMPIInstance().
In this routine after instantiating the new CMPIInstance, before
returning it has the following code

   #ifdef HAVE_DEFAULT_PROPERTIES
      if(!override) {
         instFillDefaultProperties(tInst,ns,cn);
      }
   #endif

This code will set the instance's properties to default values as defined by the CIM class.
The side effect of this is that all values of key properties (object path) of the returned instance will
be set back to null !!

I moved the instFillDefaultProperties() method to before the values of key properties of the
instance are set.  That is, initialize all properties to default values, then set the key properties.

The patch file is attached.

One way this bug occurs is when creating an instance of IndicationFilter via local connect (SfcbLocal).


Note that this bug does not occur when using http connection (cimXml)
because sfcc has its own version of instance.c  (backend/cimxml/instance.c) in which the
native_new_CMPIInstance() method does not try to set properties to default values.


----------------------------------------------------------------------

>Comment By: Michael Chase-Salerno (mchasal)
Date: 2008-11-21 15:24

Message:
File Added: 2021888.patch

----------------------------------------------------------------------

Comment By: Chris Buccella (buccella)
Date: 2008-11-14 15:00

Message:
mchasal: it would be good to write a unit test for
internal_new_CMPIInstance() before fixing this

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=712784&aid=2021888&group_id=128809