Skip to content

Commit ba3a8f5

Browse files
author
fgps
authored
fix(webclient):Fixed an error where parameters could not be assigned correctly due to incorrect form of parameters passed in
1 parent e076a08 commit ba3a8f5

5 files changed

Lines changed: 16 additions & 17 deletions

File tree

apps/webclient/packages/webclient-v2.1.2/inc/webclient.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ int webclient_get_position(struct webclient_session *session, const char *URI, i
113113
int webclient_post(struct webclient_session *session, const char *URI, const char *post_data);
114114

115115
/* close and release wenclient session */
116-
int webclient_close(struct webclient_session *session);
116+
int webclient_close(struct webclient_session **session);
117117

118118
int webclient_set_timeout(struct webclient_session *session, int millisecond);
119119

apps/webclient/packages/webclient-v2.1.2/samples/webclient_get_sample.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ static int webclient_get_comm(const char *uri)
100100
__exit:
101101
if (session)
102102
{
103-
webclient_close(session);
103+
webclient_close(&session);
104104
}
105105

106106
if (buffer)

apps/webclient/packages/webclient-v2.1.2/samples/webclient_post_sample.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ static int webclient_post_comm(const char *uri, const char *post_data)
7676
__exit:
7777
if (session)
7878
{
79-
webclient_close(session);
79+
webclient_close(&session);
8080
}
8181

8282
if (buffer)

apps/webclient/packages/webclient-v2.1.2/src/webclient.c

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1409,26 +1409,26 @@ static int webclient_clean(struct webclient_session *session)
14091409
*
14101410
* @return 0: close success
14111411
*/
1412-
int webclient_close(struct webclient_session *session)
1412+
int webclient_close(struct webclient_session **session)
14131413
{
1414-
RT_ASSERT(session);
1414+
RT_ASSERT(*session);
14151415

1416-
webclient_clean(session);
1416+
webclient_clean(*session);
14171417

1418-
if (session->header && session->header->buffer)
1418+
if ((*session)->header && (*session)->header->buffer)
14191419
{
1420-
web_free(session->header->buffer);
1420+
web_free((*session)->header->buffer);
14211421
}
14221422

1423-
if (session->header)
1423+
if ((*session)->header)
14241424
{
1425-
web_free(session->header);
1425+
web_free((*session)->header);
14261426
}
14271427

1428-
if (session)
1428+
if (*session)
14291429
{
1430-
web_free(session);
1431-
session = RT_NULL;
1430+
web_free(*session);
1431+
*session = RT_NULL;
14321432
}
14331433

14341434
return 0;
@@ -1689,8 +1689,7 @@ int webclient_request(const char *URI, const char *header, const char *post_data
16891689
__exit:
16901690
if (session)
16911691
{
1692-
webclient_close(session);
1693-
session = RT_NULL;
1692+
webclient_close(&session);
16941693
}
16951694

16961695
if (rc < 0)

apps/webclient/packages/webclient-v2.1.2/src/webclient_file.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ int webclient_get_file(const char* URI, const char* filename)
130130

131131
if (session != RT_NULL)
132132
{
133-
webclient_close(session);
133+
webclient_close(&session);
134134
}
135135

136136
if (ptr != RT_NULL)
@@ -285,7 +285,7 @@ int webclient_post_file(const char* URI, const char* filename,
285285

286286
if (session != RT_NULL)
287287
{
288-
webclient_close(session);
288+
webclient_close(&session);
289289
}
290290

291291
if (buffer != RT_NULL)

0 commit comments

Comments
 (0)