From 3c6d49a65b1928938b48866cb730d10577b496c4 Mon Sep 17 00:00:00 2001 From: Ratan Gupta Date: Wed, 16 Jan 2019 13:58:34 +0530 Subject: Fixes the method signature generation logic 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 --- module/obmc/wsgi/apps/rest_dbus.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'module/obmc') 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 -- cgit v1.2.1