esp_rmaker_params: Fixes for issues seen when sending buffer is re-allocated

Author: shahpiyushv

components/esp_rainmaker/src/core/esp_rmaker_param.c

@@ -134,9 +134,6 @@ static esp_err_t esp_rmaker_populate_params(char *buf, size_t *buf_len, uint8_t
                     device_added = true;
                 }
                 esp_rmaker_report_value(&param->val, param->name, &jstr);
-                if (reset_flags) {
-                    param->flags &= ~flags;
-                }
             }
             param = param->next;
         }
@@ -148,6 +145,23 @@ static esp_err_t esp_rmaker_populate_params(char *buf, size_t *buf_len, uint8_t
     if (json_gen_end_object(&jstr) < 0) {
         err = ESP_ERR_NO_MEM;
     }
+    /* Resetting the flags after creating the JSON in order to handle cases wherein
+     * memory has been insufficient and this same function would have to be called
+     * again with a larger buffer.
+     */
+    if (err == ESP_OK) {
+        device = esp_rmaker_node_get_first_device(esp_rmaker_get_node());
+        while (device) {
+            _esp_rmaker_param_t *param = device->params;
+            while (param) {
+                if (reset_flags) {
+                    param->flags &= ~flags;
+                }
+                param = param->next;
+            }
+            device = device->next;
+        }
+    }
     *buf_len = json_gen_str_end(&jstr);
     return err;
 }