Browse Source

jansson: rename parameter for janssonmod_get_field()

- more suggestive relation with the meaning of the field name parameter
Daniel-Constantin Mierla 1 year ago
parent
commit
3b47b56647
2 changed files with 8 additions and 8 deletions
  1. 7 7
      src/modules/jansson/jansson_funcs.c
  2. 1 1
      src/modules/jansson/jansson_funcs.h

+ 7 - 7
src/modules/jansson/jansson_funcs.c

@@ -95,22 +95,22 @@ int janssonmod_get(struct sip_msg *msg, char *path_in, char *src_in, char *dst)
 }
 
 int janssonmod_get_field(
-		struct sip_msg *msg, char *path_in, char *src_in, char *dst)
+		struct sip_msg *msg, char *field_in, char *src_in, char *dst)
 {
 	str src_s;
-	str path_s;
+	str field_s;
 
-	if(fixup_get_svalue(msg, (gparam_p)src_in, &src_s) != 0) {
-		ERR("cannot get json string value\n");
+	if(fixup_get_svalue(msg, (gparam_p)field_in, &field_s) != 0) {
+		ERR("cannot get field name string value\n");
 		return -1;
 	}
 
-	if(fixup_get_svalue(msg, (gparam_p)path_in, &path_s) != 0) {
-		ERR("cannot get path string value\n");
+	if(fixup_get_svalue(msg, (gparam_p)src_in, &src_s) != 0) {
+		ERR("cannot get json string value\n");
 		return -1;
 	}
 
-	return janssonmod_get_helper(msg, &path_s, 1, &src_s, (pv_spec_t *)dst);
+	return janssonmod_get_helper(msg, &field_s, 1, &src_s, (pv_spec_t *)dst);
 }
 
 int janssonmod_pv_get(

+ 1 - 1
src/modules/jansson/jansson_funcs.h

@@ -28,7 +28,7 @@
 int janssonmod_get(
 		struct sip_msg *msg, char *path_in, char *json_in, char *result);
 int janssonmod_get_field(
-		struct sip_msg *msg, char *path_in, char *src_in, char *dst);
+		struct sip_msg *msg, char *field_in, char *src_in, char *dst);
 int janssonmod_pv_get(
 		struct sip_msg *msg, char *path_in, char *json_in, char *result);
 int janssonmod_set(unsigned int append, struct sip_msg *msg, char *type_in,