[jira] [Closed] (CELIX-50) driverAttributes_getDriverId does not return DriverId

["Alexander Broekhuis (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CELIX-50?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alexander Broekhuis closed CELIX-50.
------------------------------------

    Resolution: Fixed
      Assignee:     (was: Alexander Broekhuis)

> driverAttributes_getDriverId does not return DriverId
> -----------------------------------------------------
>
>                 Key: CELIX-50
>                 URL: https://issues.apache.org/jira/browse/CELIX-50
>             Project: Celix
>          Issue Type: Bug
>    Affects Versions: 0.0.1-incubating, 1.0.0-incubating
>            Reporter: Bjoern Petri
>            Priority: Minor
>              Labels: device, driver
>             Fix For: 1.0.0-incubating
>
>
> I had some problems, when I was trying to publish two devices from one service and attach two drivers of the same device category to it. 
> Well, I figured out, that the deviceManager_attachAlgorithm function was not able to remove the already installed drivers from the manager->drivers list cause driverAttributes_getDriverId always returned CELIX_ENOMEM. 
> Taking a look at the code (driverAttributes_getDriverId in device_access/device_access/private/src/driver_attributes.c), I found:
> [...]
> char *id_prop = properties_get(properties, "DRIVER_ID")
> if (id_prop) {
>    status = CELIX_ENOMEM;
> }
> [...]
> which should probably be:
> [...]
> char *id_prop = properties_get(properties, "DRIVER_ID")
> if (id_prop == NULL) {
>    status = CELIX_ENOMEM;
> }
> [...]
> I would be glad if you shortly can confirm that, so I will create a patch. 
>  



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)