[Commits] [SCM] claws branch, gtk2, updated. 3.20.0-31-ge8880c469

miras at claws-mail.org miras at claws-mail.org
Tue Jan 9 10:39:58 UTC 2024


The branch, gtk2 has been updated
       via  e8880c469c51cf386e29efe79f54d8a07fd6161f (commit)
      from  f3aad9310c8c293d5ad2e929bc4c00817da0eb64 (commit)

Summary of changes:
 src/oauth2.c | 100 +++++++++++++++++++++++++++++------------------------------
 1 file changed, 50 insertions(+), 50 deletions(-)


- Log -----------------------------------------------------------------
commit e8880c469c51cf386e29efe79f54d8a07fd6161f
Author: Michael Rasmussen <mir at datanom.net>
Date:   Tue Jan 9 11:37:41 2024 +0100

    Fix bug #Bug 4730. Patch from Olaf Hering
    
    Signed-off-by: Michael Rasmussen <mir at datanom.net>

diff --git a/src/oauth2.c b/src/oauth2.c
index 43da8b9d5..e8bd794a2 100644
--- a/src/oauth2.c
+++ b/src/oauth2.c
@@ -129,7 +129,6 @@ static gchar *OAUTH2CodeMarker[5][2] = {
 static gint oauth2_post_request (gchar *buf, gchar *host, gchar *resource, gchar *header, gchar *body);
 static gint oauth2_filter_refresh (gchar *json, gchar *refresh_token);
 static gint oauth2_filter_access (gchar *json, gchar *access_token, gint *expiry);
-static gint oauth2_contact_server (SockInfo *sock, gchar *request, gchar *response);
 
 
 static gint oauth2_post_request (gchar *buf, gchar *host, gchar *resource, gchar *header, gchar *body)
@@ -214,6 +213,50 @@ static gchar* oauth2_get_token_from_response(Oauth2Service provider, const gchar
 	return token;
 }
 
+static gchar *oauth2_contact_server(SockInfo *sock, const gchar *request)
+{
+	gboolean got_some_error, timeout;
+	gint ret;
+	char buf[1024];
+	GString *response = g_string_sized_new(sizeof(buf));
+	time_t end_time = time(NULL);
+
+	end_time += prefs_common_get_prefs()->io_timeout_secs;
+
+	if (!response)
+		return NULL;
+
+	if (sock_write(sock, request, strlen(request)) < 0) {
+		log_message(LOG_PROTOCOL, _("OAuth2 socket write error\n"));
+		return NULL;
+	}
+
+	do {
+		ret = sock_read(sock, buf, sizeof(buf) - 1);
+		got_some_error = ret < 0;
+		timeout = time(NULL) > end_time;
+
+		if (timeout)
+			break;
+
+		if (ret < 0 && errno == EAGAIN)
+			continue;
+
+		if (got_some_error)
+			break;
+
+		if (ret) {
+			buf[ret] = '\0';
+			g_string_append_len(response, buf, ret);
+		}
+	} while (ret);
+
+	if (timeout)
+		log_message(LOG_PROTOCOL, _("OAuth2 socket timeout error\n"));
+
+	return g_string_free(response, got_some_error || timeout);
+}
+
 int oauth2_obtain_tokens (Oauth2Service provider, OAUTH2Data *OAUTH2Data, const gchar *authcode)
 {
 	gchar *request;
@@ -266,7 +309,6 @@ int oauth2_obtain_tokens (Oauth2Service provider, OAUTH2Data *OAUTH2Data, const
 	refresh_token = g_malloc(OAUTH2BUFSIZE+1);	
 	access_token = g_malloc(OAUTH2BUFSIZE+1);
 	request = g_malloc(OAUTH2BUFSIZE+1);
-	response = g_malloc0(OAUTH2BUFSIZE+1);
 
 	if(OAUTH2Data->custom_client_id)
 	  client_id = g_strdup(OAUTH2Data->custom_client_id);
@@ -330,9 +372,9 @@ int oauth2_obtain_tokens (Oauth2Service provider, OAUTH2Data *OAUTH2Data, const
 
         debug_print("Complete body: %s\n", body);
 	oauth2_post_request (request, OAUTH2info[i][OA2_BASE_URL], OAUTH2info[i][OA2_ACCESS_RESOURCE], header, body);
-	ret = oauth2_contact_server (sock, request, response);
+	response = oauth2_contact_server (sock, request);
 
-	if(oauth2_filter_access (response, access_token, &expiry) == 0){
+	if(response && oauth2_filter_access (response, access_token, &expiry) == 0){
 	  OAUTH2Data->access_token = g_strdup(access_token);
 	  OAUTH2Data->expiry = expiry;
 	  OAUTH2Data->expiry_str = g_strdup_printf ("%i", expiry);
@@ -344,7 +386,7 @@ int oauth2_obtain_tokens (Oauth2Service provider, OAUTH2Data *OAUTH2Data, const
 	  ret = 1;
 	}
 
-	if(oauth2_filter_refresh (response, refresh_token) == 0){
+	if(response && oauth2_filter_refresh (response, refresh_token) == 0){
 	  OAUTH2Data->refresh_token = g_strdup(refresh_token);
 	  log_message(LOG_PROTOCOL, _("OAuth2 refresh token obtained\n"));
 	}else{
@@ -407,7 +449,6 @@ gint oauth2_use_refresh_token (Oauth2Service provider, OAUTH2Data *OAUTH2Data)
 	access_token = g_malloc(OAUTH2BUFSIZE+1);
 	refresh_token = g_malloc(OAUTH2BUFSIZE+1);
 	request = g_malloc(OAUTH2BUFSIZE+1);
-	response = g_malloc(OAUTH2BUFSIZE+1);
 
 	if(OAUTH2Data->custom_client_id)
 	  client_id = g_strdup(OAUTH2Data->custom_client_id);
@@ -466,9 +507,9 @@ gint oauth2_use_refresh_token (Oauth2Service provider, OAUTH2Data *OAUTH2Data)
 	}
 
 	oauth2_post_request (request, OAUTH2info[i][OA2_BASE_URL], OAUTH2info[i][OA2_REFRESH_RESOURCE], header, body);
-	ret = oauth2_contact_server (sock, request, response);
+	response = oauth2_contact_server (sock, request);
 
-	if(oauth2_filter_access (response, access_token, &expiry) == 0){
+	if(response && oauth2_filter_access (response, access_token, &expiry) == 0){
 	  OAUTH2Data->access_token = g_strdup(access_token);
 	  OAUTH2Data->expiry = expiry;
 	  OAUTH2Data->expiry_str = g_strdup_printf ("%i", expiry);
@@ -480,7 +521,7 @@ gint oauth2_use_refresh_token (Oauth2Service provider, OAUTH2Data *OAUTH2Data)
 	  ret = 1;
 	}
 
-	if (oauth2_filter_refresh (response, refresh_token) == 0) {
+	if (response && oauth2_filter_refresh (response, refresh_token) == 0) {
 		OAUTH2Data->refresh_token = g_strdup(refresh_token);
 		log_message(LOG_PROTOCOL, _("OAuth2 replacement refresh token provided\n"));
 	} else
@@ -502,47 +543,6 @@ gint oauth2_use_refresh_token (Oauth2Service provider, OAUTH2Data *OAUTH2Data)
 	return (ret);
 }
 
-static gint oauth2_contact_server (SockInfo *sock, gchar *request, gchar *response)
-{
-	gint ret;
-	gchar *token;
-	gint toread = OAUTH2BUFSIZE;	
-	time_t startplus = time(NULL);
-	gchar *tmp;
-	
-	gint timeout_secs = prefs_common_get_prefs()->io_timeout_secs;
-	startplus += timeout_secs;
-	
-	if (sock_write (sock, request, strlen(request)) < 0) {
-	  log_message(LOG_PROTOCOL, _("OAuth2 socket write error\n"));
-	  return (1);
-	}
-	
-	token = g_strconcat ("", NULL);
-	do {
-	  
-	  ret = sock_read  (sock, response, OAUTH2BUFSIZE);
-	  if (ret < 0 && errno == EAGAIN)
-	    continue;
-	  if (ret < 0) 
-	    break;
-	  if (ret == 0)
-	    break;
-	  
-	  toread -= ret;
-	  tmp = g_strconcat(token, response, NULL);
-	  g_free(token);
-	  token = tmp;
-	} while ((toread > 0) && (time(NULL) < startplus)); 
-	
-	if(time(NULL) >= startplus)
-	  log_message(LOG_PROTOCOL, _("OAuth2 socket timeout error\n"));
-	
-	g_free(token);
-	
-	return (0);
-}
-
 gint oauth2_authorisation_url (Oauth2Service provider, gchar **url, const gchar *custom_client_id)
 {
 	gint i;

-----------------------------------------------------------------------


hooks/post-receive
-- 
Claws Mail


More information about the Commits mailing list