[PATCH] radio-settings: Show all available technologies

[PATCH] radio-settings: Show all available technologies

[Alfonso Sanchez-Beato]

[-- Attachment #1: Type: text/plain, Size: 793 bytes --]

Not all possible futures technologies were being showed on the DBus
interface.
---
 src/radio-settings.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/radio-settings.c b/src/radio-settings.c
index 0a49305..1a511dc 100644
--- a/src/radio-settings.c
+++ b/src/radio-settings.c
@@ -225,12 +225,12 @@ static DBusMessage *radio_get_properties_reply(DBusMessage *msg,
 	}
 
 	if (rs->available_rats) {
-		const char *rats[sizeof(uint32_t) + 1];
+		const char *rats[sizeof(uint32_t) * CHAR_BIT + 1];
 		const char **dbus_rats = rats;
 		int n = 0;
 		unsigned int i;
 
-		for (i = 0; i < sizeof(uint32_t); i++) {
+		for (i = 0; i < sizeof(uint32_t) * CHAR_BIT; i++) {
 			int tech = 1 << i;
 
 			if (!(rs->available_rats & tech))
-- 
2.1.0

Re: [PATCH] radio-settings: Show all available technologies

[Denis Kenzior]

[-- Attachment #1: Type: text/plain, Size: 324 bytes --]

Hi Alfonso,

On 12/12/2014 05:25 AM, Alfonso Sanchez-Beato wrote:
> Not all possible futures technologies were being showed on the DBus
> interface.
> ---
>   src/radio-settings.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>

Good catch.

Patch has been applied, thanks.

Regards,
-Denis