X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fafb-session.c;h=bb10c2cbd5b91ffd53f2c1c36441489fea979db3;hb=refs%2Fchanges%2F49%2F23149%2F1;hp=1bdb59db441a3e03e0e73964147e17b681403b30;hpb=6fe59e27c0640462f92133106b4d4d52d77683f9;p=src%2Fapp-framework-binder.git

diff --git a/src/afb-session.c b/src/afb-session.c
index 1bdb59db..bb10c2cb 100644
--- a/src/afb-session.c
+++ b/src/afb-session.c
@@ -28,6 +28,7 @@
 
 #include "afb-session.h"
 #include "afb-hook.h"
+#include "afb-token.h"
 #include "verbose.h"
 #include "pearson.h"
 #include "uuid.h"
@@ -68,7 +69,7 @@ struct afb_session
 	uint8_t autoclose: 1;   /**< close the session when unreferenced */
 	uint8_t notinset: 1;	/**< session removed from the set of sessions */
 	uuid_stringz_t uuid;    /**< long term authentication of remote client */
-	uuid_stringz_t token;   /**< short term authentication of remote client */
+	struct afb_token *token;/**< short term authentication of remote client */
 };
 
 /**
@@ -79,14 +80,14 @@ static struct {
 	int max;                /**< maximum count of sessions */
 	int timeout;            /**< common initial timeout */
 	struct afb_session *heads[HEADCOUNT]; /**< sessions */
-	uuid_stringz_t initok;  /**< common initial token */
+	struct afb_token *initok;/**< common initial token */
 	pthread_mutex_t mutex;  /**< declare a mutex to protect hash table */
 } sessions = {
 	.count = 0,
 	.max = 10,
 	.timeout = 3600,
 	.heads = { 0 },
-	.initok = { 0 },
+	.initok = 0,
 	.mutex = PTHREAD_MUTEX_INITIALIZER
 };
 
@@ -203,6 +204,7 @@ static void session_destroy (struct afb_session *session)
 	afb_hook_session_destroy(session);
 #endif
 	pthread_mutex_destroy(&session->mutex);
+	afb_token_unref(session->token);
 	free(session->lang);
 	free(session);
 }
@@ -249,12 +251,13 @@ static struct afb_session *session_add(const char *uuid, int timeout, time_t now
 	pthread_mutex_init(&session->mutex, NULL);
 	session->refcount = 1;
 	strcpy(session->uuid, uuid);
-	strcpy(session->token, sessions.initok);
+	session->token = afb_token_addref(sessions.initok);
 	session->timeout = timeout;
 	session_update_expiration(session, now);
 
 	/* add */
 	if (sessionset_add(session, hashidx)) {
+		afb_token_unref(session->token);
 		free(session);
 		return NULL;
 	}
@@ -309,6 +312,9 @@ static time_t sessionset_cleanup (int force)
  */
 int afb_session_init (int max_session_count, int timeout, const char *initok)
 {
+	int rc;
+	uuid_stringz_t uuid;
+
 	/* check parameters */
 	if (initok && strlen(initok) >= sizeof sessions.initok) {
 		ERROR("initial token '%s' too long (max length %d)",
@@ -322,10 +328,16 @@ int afb_session_init (int max_session_count, int timeout, const char *initok)
 	sessionset_cleanup(1);
 	sessions.max = max_session_count;
 	sessions.timeout = timeout;
-	if (initok == NULL)
-		uuid_new_stringz(sessions.initok);
-	else
-		strcpy(sessions.initok, initok);
+	if (initok == NULL) {
+		uuid_new_stringz(uuid);
+		initok = uuid;
+	}
+	sessions.initok = 0;
+	if (*initok) {
+		rc = afb_token_get(&sessions.initok, initok);
+		if (rc < 0)
+			return rc;
+	}
 	sessionset_unlock();
 	return 0;
 }
@@ -367,7 +379,7 @@ void afb_session_purge()
  */
 const char *afb_session_initial_token()
 {
-	return sessions.initok;
+	return sessions.initok ? afb_token_string(sessions.initok) : "";
 }
 
 /* Searchs the session of 'uuid' */
@@ -528,35 +540,17 @@ int afb_session_check_token (struct afb_session *session, const char *token)
 	session_lock(session);
 	r = !session->closed
 	  && session->expiration >= NOW
-	  && !(session->token[0] && strcmp (token, session->token));
+	  && !(session->token && strcmp(token, afb_token_string(session->token)));
 	session_unlock(session);
 	return r;
 }
 
-/* generate a new token and update client context */
-void afb_session_new_token (struct afb_session *session)
-{
-	session_lock(session);
-	uuid_new_stringz(session->token);
-	session_update_expiration(session, NOW);
-#if WITH_AFB_HOOK
-	afb_hook_session_renew(session);
-#endif
-	session_unlock(session);
-}
-
 /* Returns the uuid of 'session' */
 const char *afb_session_uuid (struct afb_session *session)
 {
 	return session->uuid;
 }
 
-/* Returns the token of 'session' */
-const char *afb_session_token (struct afb_session *session)
-{
-	return session->token;
-}
-
 /**
  * Get the index of the 'key' in the cookies array.
  * @param key the key to scan
@@ -579,7 +573,7 @@ static int cookeyidx(const void *key)
  * @param makecb	the creation function or NULL
  * @param freecb	the release function or NULL
  * @param closure	an argument for makecb or the value if makecb==NULL
- * @param replace	a boolean enforcing replecement of the previous value
+ * @param replace	a boolean enforcing replacement of the previous value
  *
  * @return the value of the cookie
  *