Merge "Fixed the unicast address for send indication" into tizen accepted/tizen/unified/20190212.084048 submit/tizen/20190211.065813 submit/tizen/20190211.100729
authorPyun DoHyun <dh79.pyun@samsung.com>
Mon, 11 Feb 2019 06:57:05 +0000 (06:57 +0000)
committerGerrit Code Review <gerrit@review.ap-northeast-2.compute.internal>
Mon, 11 Feb 2019 06:57:05 +0000 (06:57 +0000)
1  2 
bt-oal/bluez_hal/src/bt-hal-gatt-server.c

@@@ -737,16 -737,11 +737,16 @@@ static void __bt_gatt_manager_method_ca
                                for (i = 0; i < char_info->flags_length; i++) {
                                        g_variant_builder_add(builder2, "s",
                                                        char_info->char_flags[i]);
 -                                      if (strncmp(char_info->char_flags[i], "write-without-response", 16) == 0)
 +
 +                                      if (strncmp(char_info->char_flags[i], "write-without-response", 16) == 0) {
 +                                              DBG("setting WriteAcquired property");
                                                writeAcquired = TRUE;
 +                                      }
  
 -                                      if (strncmp(char_info->char_flags[i], "notify", 4) == 0)
 +                                      if (strncmp(char_info->char_flags[i], "notify", 4) == 0) {
 +                                              DBG("setting NotifyAcquired property");
                                                notifyAcquired = TRUE;
 +                                      }
                                }
  
                                flags_val = g_variant_new("as", builder2);
                                g_variant_builder_add(inner_builder, "{sv}", "Notifying",
                                                g_variant_new("b", notify));
  
 -                              /* Unicast */
 -                              g_variant_builder_add(inner_builder, "{sv}", "WriteAcquired",
 +                              /* WriteAcquired */
 +                              if (writeAcquired == TRUE) {
 +                                      DBG("adding WriteAcquired property");
 +                                      g_variant_builder_add(inner_builder, "{sv}", "WriteAcquired",
                                                                        g_variant_new("b", writeAcquired));
 +                              }
  
                                /* NotifyAcquired */
 -                                                      g_variant_builder_add(inner_builder, "{sv}", "NotifyAcquired",
 -                                                                      g_variant_new("b", notifyAcquired));
 +                              if (notifyAcquired == TRUE) {
 +                                      DBG("adding NotifyAcquired property");
 +                                      g_variant_builder_add(inner_builder, "{sv}", "NotifyAcquired",
 +                                                      g_variant_new("b", notifyAcquired));
 +                              }
  
                                /* Unicast */
                                unicast = g_strdup("00:00:00:00:00:00");
@@@ -2895,7 -2884,8 +2895,8 @@@ static bt_status_t gatt_server_send_ind
        g_variant_builder_add(outer_builder, "{sv}", "Notifying",
                        g_variant_new("b", notify));
  
-       _bt_hal_convert_addr_type_to_string(addr, (unsigned char *)conn_info->addr);
+       memcpy(addr, conn_info->addr, BT_HAL_ADDRESS_STRING_SIZE);
+       DBG("Send Indication to Unicast addr [%s]", addr);
  
        g_variant_builder_add(outer_builder, "{sv}", "Unicast",
                        g_variant_new("s", addr));