Просмотр исходного кода

Merge branch 'prov/protocomm_nimble_strdup_remove' into 'master'

Provisioning: Remove unnecessary strdup/free from protocomm_nimble

Closes IDFGH-4980

See merge request espressif/esp-idf!12889
Mahavir Jain 4 лет назад
Родитель
Сommit
d6a6c3b46d
1 измененных файлов с 2 добавлено и 8 удалено
  1. 2 8
      components/protocomm/src/transports/protocomm_nimble.c

+ 2 - 8
components/protocomm/src/transports/protocomm_nimble.c

@@ -284,15 +284,9 @@ gatt_svr_dsc_access(uint16_t conn_handle, uint16_t attr_handle, struct
     }
 
     int rc;
-    char *temp_outbuf = strdup(ctxt->dsc->arg);
-    if (temp_outbuf == NULL) {
-        ESP_LOGE(TAG, "Error duplicating user description of characteristic");
-        return BLE_ATT_ERR_INSUFFICIENT_RES;
-    }
+    ssize_t temp_outlen = strlen(ctxt->dsc->arg);
 
-    ssize_t temp_outlen = strlen(temp_outbuf);
-    rc = os_mbuf_append(ctxt->om, temp_outbuf, temp_outlen);
-    free(temp_outbuf);
+    rc = os_mbuf_append(ctxt->om, ctxt->dsc->arg, temp_outlen);
     return rc;
 }