Mercurial > hg > freeDiameter
changeset 708:de2c260b6f6b
Fix remaining test testappacct
author | Sebastien Decugis <sdecugis@nict.go.jp> |
---|---|
date | Thu, 10 Feb 2011 10:24:11 +0900 |
parents | e387d5c6b6f5 |
children | 19a9470de77a |
files | tests/testappacct.c |
diffstat | 1 files changed, 17 insertions(+), 13 deletions(-) [+] |
line wrap: on
line diff
--- a/tests/testappacct.c Wed Feb 09 18:08:54 2011 +0900 +++ b/tests/testappacct.c Thu Feb 10 10:24:11 2011 +0900 @@ -84,6 +84,9 @@ return 0; } +#define LOCAL_ID "test.app.acct" +#define LOCAL_REALM "app.acct" + /* Main test routine */ int main(int argc, char *argv[]) { @@ -91,14 +94,15 @@ extern int fd_ext_init(int major, int minor, char * conffile); /* defined in include's extension.h */ extern void fd_ext_fini(void); /* defined in the extension itself */ struct msg * msg; - char * sess_bkp; + os0_t sess_bkp; + size_t sess_bkp_len; /* First, initialize the daemon modules */ INIT_FD(); - fd_g_config->cnf_diamid = strdup("test.app.acct"); - fd_g_config->cnf_diamid_len = strlen(fd_g_config->cnf_diamid); - fd_g_config->cnf_diamrlm = strdup("app.acct"); - fd_g_config->cnf_diamrlm_len = strlen(fd_g_config->cnf_diamrlm); + fd_g_config->cnf_diamid = strdup(LOCAL_ID); + fd_g_config->cnf_diamid_len = CONSTSTRLEN(LOCAL_ID); + fd_g_config->cnf_diamrlm = strdup(LOCAL_REALM); + fd_g_config->cnf_diamrlm_len = CONSTSTRLEN(LOCAL_REALM); CHECK( 0, fd_queues_init() ); CHECK( 0, fd_msg_init() ); @@ -165,16 +169,16 @@ /* sid */ { struct session * sess = NULL; - char * s; - CHECK( 0, fd_sess_new( &sess, fd_g_config->cnf_diamid, NULL, 0) ); - CHECK( 0, fd_sess_getsid(sess, &s) ); - sess_bkp = strdup(s); + os0_t s; + CHECK( 0, fd_sess_new( &sess, fd_g_config->cnf_diamid, fd_g_config->cnf_diamid_len, NULL, 0) ); + CHECK( 0, fd_sess_getsid(sess, &s, &sess_bkp_len) ); + CHECK( 1, (sess_bkp = os0dup(s, sess_bkp_len)) ? 1 : 0); CHECK( 0, fd_dict_search ( fd_g_config->cnf_dict, DICT_AVP, AVP_BY_NAME, "Session-Id", &d, ENOENT ) ); CHECK( 0, fd_msg_avp_new ( d, 0, &avp ) ); memset(&avp_val, 0, sizeof(avp_val)); - avp_val.os.data = (unsigned char *)sess_bkp; - avp_val.os.len = strlen(sess_bkp); + avp_val.os.data = sess_bkp; + avp_val.os.len = sess_bkp_len; CHECK( 0, fd_msg_avp_setvalue ( avp, &avp_val ) ); CHECK( 0, fd_msg_avp_add ( msg, MSG_BRW_FIRST_CHILD, avp) ); } @@ -232,7 +236,7 @@ } /* Source */ - CHECK( 0, fd_msg_source_set( msg, "peer3", 1, fd_g_config->cnf_dict ) ); + CHECK( 0, fd_msg_source_set( msg, "peer3", CONSTSTRLEN("peer3"), 1, fd_g_config->cnf_dict ) ); } /* Now, have the daemon handle this */ @@ -252,7 +256,7 @@ /* We also check that the Session-Id we retrieve is the same as what we generated earlier (not trashed in the process) */ s = PQgetvalue(res, 0, 0); - CHECK( 0, strcmp(s, sess_bkp) ); + CHECK( 0, strcmp(s, (char *)sess_bkp) ); PQclear(res); }