Improved logging

sesman/scp_v0.c

@@ -42,18 +42,29 @@ scp_v0_process(struct SCP_CONNECTION* c, struct SCP_SESSION* s)
   {
     /* this is just authentication in a gateway situation */
     /* g_writeln("SCP_GW_AUTHENTICATION message received"); */
-      if(data){	 
+    if(data)
+    {
 	    if (1 == access_login_allowed(s->username))
 	    {
 	      /* the user is member of the correct groups. */
 	      scp_v0s_replyauthentication(c,0);
+        log_message( LOG_LEVEL_INFO,"Access permitted for user: %s",
+          s->username);
 	      /* g_writeln("Connection allowed"); */
-	  }else{	      
+	    }
+      else
+      {
         scp_v0s_replyauthentication(c,3);
+        log_message( LOG_LEVEL_INFO,"Username okey but group problem for user: %s",
+          s->username);
 	      /* g_writeln("user password ok, but group problem"); */
 	    }
-      }else{	  
+    }
+    else
+    {
 	    /* g_writeln("username or password error"); */
+      log_message( LOG_LEVEL_INFO,"Username or password error for user: %s",
+        s->username);
       scp_v0s_replyauthentication(c,2);	  
     }
     auth_end(data);
@@ -66,11 +77,13 @@ scp_v0_process(struct SCP_CONNECTION* c, struct SCP_SESSION* s)
       display = s_item->display;
       if (0 != s->client_ip)
       {
-        log_message( LOG_LEVEL_INFO, "++ reconnected session: username %s, display :%d.0, session_pid %d, ip %s", s->username, display, s_item->pid, s->client_ip);
+        log_message( LOG_LEVEL_INFO, "++ reconnected session: username %s, display :%d.0, "
+          "session_pid %d, ip %s", s->username, display, s_item->pid, s->client_ip);
       }
       else
       {
-        log_message(LOG_LEVEL_INFO, "++ reconnected session: username %s, display :%d.0, session_pid %d", s->username, display, s_item->pid);
+        log_message(LOG_LEVEL_INFO, "++ reconnected session: username %s, display :%d.0, "
+          "session_pid %d", s->username, display, s_item->pid);
       }
       auth_end(data);
       /* don't set data to null here */
@@ -82,11 +95,13 @@ scp_v0_process(struct SCP_CONNECTION* c, struct SCP_SESSION* s)
       {
         if (0 != s->client_ip)
         {
-          log_message(LOG_LEVEL_INFO, "++ created session (access granted): username %s, ip %s", s->username, s->client_ip);
+          log_message(LOG_LEVEL_INFO, "++ created session (access granted): "
+            "username %s, ip %s", s->username, s->client_ip);
         }
         else
         {
-          log_message(LOG_LEVEL_INFO, "++ created session (access granted): username %s", s->username);
+          log_message(LOG_LEVEL_INFO, "++ created session (access granted): "
+            "username %s", s->username);
         }
 
         if (SCP_SESSION_TYPE_XVNC == s->type)

xrdp/xrdp_mm.c

@@ -21,6 +21,7 @@
 */
 
 #include "xrdp.h"
+#include "log.h"
 #define ACCESS
 
 /*****************************************************************************/