diff options
| author | Ratan Gupta <ratagupt@linux.vnet.ibm.com> | 2019-01-16 13:58:34 +0530 |
|---|---|---|
| committer | Ratan Gupta <ratagupt@linux.vnet.ibm.com> | 2019-01-16 14:02:01 +0530 |
| commit | 3c6d49a65b1928938b48866cb730d10577b496c4 (patch) | |
| tree | 346fb164cc34dc71561873166489a6b64105a2b8 /module/obmc/wsgi/apps | |
| parent | 44573ab21aa463504344d311396e5c365b8ea41a (diff) | |
| download | phosphor-rest-server-3c6d49a65b1928938b48866cb730d10577b496c4.tar.gz phosphor-rest-server-3c6d49a65b1928938b48866cb730d10577b496c4.zip | |
During generation of the method signature, Both Input and Output argument
is being used, where only Input argument should have been considered.
This method signature is required as we convert the input values into its actual
types by introspecting the interface.
https://github.com/openbmc/openbmc/issues/1160
Fixes openbmc/openbmc#3453
Change-Id: I3eb99736295b76176a9964f7542f326007e1c4a4
Signed-off-by: Ratan Gupta <ratagupt@linux.vnet.ibm.com>
Diffstat (limited to 'module/obmc/wsgi/apps')
| -rw-r--r-- | module/obmc/wsgi/apps/rest_dbus.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/module/obmc/wsgi/apps/rest_dbus.py b/module/obmc/wsgi/apps/rest_dbus.py index a26532a..8df9805 100644 --- a/module/obmc/wsgi/apps/rest_dbus.py +++ b/module/obmc/wsgi/apps/rest_dbus.py @@ -97,7 +97,8 @@ def get_method_signature(bus, service, object_path, interface, method): for dbus_method in dbus_intf.findall('method'): if(dbus_method.get('name') == method): for arg in dbus_method.findall('arg'): - arglist.append(arg.get('type')) + if (arg.get('direction') == 'in'): + arglist.append(arg.get('type')) return arglist |
