Merge pull request #584 from fengzeroz/main

check tag name/address len

ft/resource/error.resource

@@ -36,6 +36,9 @@ ${NEU_ERR_TAG_NAME_CONFLICT}           2202
 ${NEU_ERR_TAG_ATTRIBUTE_NOT_SUPPORT}   2203
 ${NEU_ERR_TAG_TYPE_NOT_SUPPORT}        2204
 ${NEU_ERR_TAG_ADDRESS_FORMAT_INVALID}  2205
+${NEU_ERR_TAG_NAME_TOO_LONG}           2206
+${NEU_ERR_TAG_ADDRESS_TOO_LONG}        2207
+${NEU_ERR_TAG_DESCRIPTION_TOO_LONG}    2208
 
 ${NEU_ERR_LIBRARY_NOT_FOUND}             2301
 ${NEU_ERR_LIBRARY_INFO_INVALID}          2302

include/neuron/errcodes.h

@@ -65,6 +65,9 @@ typedef enum {
     NEU_ERR_TAG_ATTRIBUTE_NOT_SUPPORT  = 2203,
     NEU_ERR_TAG_TYPE_NOT_SUPPORT       = 2204,
     NEU_ERR_TAG_ADDRESS_FORMAT_INVALID = 2205,
+    NEU_ERR_TAG_NAME_TOO_LONG          = 2206,
+    NEU_ERR_TAG_ADDRESS_TOO_LONG       = 2207,
+    NEU_ERR_TAG_DESCRIPTION_TOO_LONG   = 2208,
 
     NEU_ERR_LIBRARY_NOT_FOUND            = 2301,
     NEU_ERR_LIBRARY_INFO_INVALID         = 2302,

src/adapter/driver/driver.c

@@ -415,6 +415,18 @@ int neu_adapter_driver_add_tag(neu_adapter_driver_t *driver, const char *group,
     int      ret  = NEU_ERR_SUCCESS;
     group_t *find = NULL;
 
+    if (strlen(tag->name) > NEU_TAG_NAME_LEN) {
+        return NEU_ERR_TAG_NAME_TOO_LONG;
+    }
+
+    if (strlen(tag->address) > NEU_TAG_ADDRESS_LEN) {
+        return NEU_ERR_TAG_ADDRESS_TOO_LONG;
+    }
+
+    if (strlen(tag->description) > NEU_TAG_DESCRIPTION_LEN) {
+        return NEU_ERR_TAG_DESCRIPTION_TOO_LONG;
+    }
+
     ret = driver->adapter.module->intf_funs->driver.validate_tag(
         driver->adapter.plugin, tag);
     if (ret != NEU_ERR_SUCCESS) {
@@ -461,6 +473,18 @@ int neu_adapter_driver_update_tag(neu_adapter_driver_t *driver,
     int      ret  = NEU_ERR_SUCCESS;
     group_t *find = NULL;
 
+    if (strlen(tag->name) > NEU_TAG_NAME_LEN) {
+        return NEU_ERR_TAG_NAME_TOO_LONG;
+    }
+
+    if (strlen(tag->address) > NEU_TAG_ADDRESS_LEN) {
+        return NEU_ERR_TAG_ADDRESS_TOO_LONG;
+    }
+
+    if (strlen(tag->description) > NEU_TAG_DESCRIPTION_LEN) {
+        return NEU_ERR_TAG_DESCRIPTION_TOO_LONG;
+    }
+
     ret = driver->adapter.module->intf_funs->driver.validate_tag(
         driver->adapter.plugin, tag);
     if (ret != NEU_ERR_SUCCESS) {