incubator-celix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexander Broekhuis (JIRA)" <j...@apache.org>
Subject [jira] [Closed] (CELIX-50) driverAttributes_getDriverId does not return DriverId
Date Tue, 17 Dec 2013 11:29:07 GMT

     [ 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)

Mime
View raw message