Fixed use of SAK in lock process when autologin is used in TDM.

This resolves bug 2945.

Signed-off-by: Michele Calgaro <michele.calgaro@yahoo.it>

5 files changed, 31 insertions, 26 deletions

diff --git a/tdm/backend/CMakeLists.txt b/tdm/backend/CMakeLists.txt
index da3f3ac03..bece8be72 100644
--- a/tdm/backend/CMakeLists.txt
+++ b/tdm/backend/CMakeLists.txt
@@ -15,6 +15,7 @@ include_directories(
   ${CMAKE_CURRENT_BINARY_DIR}
   ${CMAKE_BINARY_DIR}
   ${DBUS_INCLUDE_DIRS}
+  ${CMAKE_SOURCE_DIR}/tdm/kfrontend
 )
 
 link_directories(
@@ -27,7 +28,7 @@ add_custom_command( OUTPUT config.ci
   COMMAND perl -w ${CMAKE_SOURCE_DIR}/tdm/confproc.pl ${CMAKE_SOURCE_DIR}/tdm/config.def config.ci
   DEPENDS ${CMAKE_SOURCE_DIR}/tdm/confproc.pl ${CMAKE_SOURCE_DIR}/tdm/config.def )
 
-set_property( SOURCE auth.c APPEND PROPERTY OBJECT_DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/config.ci )
+set_property( SOURCE auth.c session.c APPEND PROPERTY OBJECT_DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/config.ci )
 
 if( WITH_XDMCP )
   set( XDMCP_LIBRARIES "Xdmcp" )
diff --git a/tdm/backend/dm.c b/tdm/backend/dm.c
index 861037032..605f0b1c4 100644
--- a/tdm/backend/dm.c
+++ b/tdm/backend/dm.c
@@ -576,12 +576,12 @@ StartRemoteLogin( struct display *d )
 		/* Let's try again with some standard paths */
 		argv[0] = (char *)realloc(argv[0], strlen("/usr/X11R6/bin/X") + 1);
 		if (argv[0] != NULL) {
-			argv[0] = "/usr/X11R6/bin/X";
+			strcpy(argv[0], "/usr/X11R6/bin/X");
 			Debug( "exec %\"[s\n", argv );
 			(void)execv( argv[0], argv );
 			LogError( "X server %\"s cannot be executed\n", argv[0] );
 		
-			argv[0] = "/usr/bin/X"; /* Shorter than the previous file name */
+			strcpy(argv[0], "/usr/bin/X"); // Shorter than the previous file name
 			Debug( "exec %\"[s\n", argv );
 			(void)execv( argv[0], argv );
 			LogError( "X server %\"s cannot be executed\n", argv[0] );
diff --git a/tdm/backend/session.c b/tdm/backend/session.c
index 4b5f372db..0bcd92d5f 100644
--- a/tdm/backend/session.c
+++ b/tdm/backend/session.c
@@ -34,6 +34,8 @@ from the copyright holder.
  * subdaemon event loop, etc.
  */
 
+
+#include "tdmconfig.h"
 #include "dm.h"
 #include "dm_error.h"
 
@@ -49,6 +51,8 @@ from the copyright holder.
 #include "consolekit.h"
 #endif
 
+#define TSAK_FIFO_FILE "/tmp/tdesocket-global/tsak"
+
 struct display *td;
 const char *td_setup = "auto";
 
@@ -576,6 +580,27 @@ ManageSession( struct display *d )
 		goto regreet;
 	}
 
+	int start_tsak = 0;
+#ifdef BUILD_TSAK
+  start_tsak = d->useSAK;
+#endif
+  if (start_tsak) {
+		if (system(KDE_BINDIR "/tsak checkdeps") != 0) {
+			start_tsak = 0;
+		}
+	}
+  if (start_tsak) {
+		int ret_pid = fork();
+		if (ret_pid == 0) {
+			// Child process
+			system(KDE_BINDIR "/tsak");
+			exit(0);
+		}
+	}
+	else {
+		remove(TSAK_FIFO_FILE);
+	}
+
 	tdiff = time( 0 ) - td->hstent->lastExit - td->openDelay;
 	if (AutoLogon( tdiff )) {
 		if (!Verify( conv_auto, FALSE ))
diff --git a/tdm/config.def b/tdm/config.def
index 124887f20..5cfc56ed5 100644
--- a/tdm/config.def
+++ b/tdm/config.def
@@ -2011,10 +2011,11 @@ Description:
 Key: UseSAK
 Type: bool
 Default: false
+User: core
 User: greeter
 Instance: #:*/false
 Comment:
- SAK
+ Use SAK
 Description:
  If true then the SAK anti-spoofing dialog will be utilized
 
diff --git a/tdm/kfrontend/kgapp.cpp b/tdm/kfrontend/kgapp.cpp
index 67178cc05..5487269d7 100644
--- a/tdm/kfrontend/kgapp.cpp
+++ b/tdm/kfrontend/kgapp.cpp
@@ -67,8 +67,6 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
 
 #include <pwd.h>
 
-#define TSAK_FIFO_FILE "/tmp/tdesocket-global/tsak"
-
 bool argb_visual_available = false;
 bool has_twin = false;
 bool is_themed = false;
@@ -217,7 +215,6 @@ kg_main( const char *argv0 )
 	TDEApplication::disableAutoDcopRegistration();
 	TDECrash::setSafer( true );
 
-	TDEProcess *tsak = 0;
 	TDEProcess *kbdl = 0;
 	TDEProcess *ccsm = 0;
 	TDEProcess *proc = 0;
@@ -230,25 +227,6 @@ kg_main( const char *argv0 )
 #else
 	trinity_desktop_lock_use_sak = false;
 #endif
- 	if (trinity_desktop_lock_use_sak) {
-		if (system(KDE_BINDIR "/tsak checkdeps") != 0) {
-			trinity_desktop_lock_use_sak = false;
-		}
-	}
-	if (trinity_desktop_lock_use_sak) {
-		tsak = new TDEProcess;
-		*tsak << TQCString( argv0, strrchr( argv0, '/' ) - argv0 + 2 ) + "tsak";
-		tsak->start(TDEProcess::Block, TDEProcess::AllOutput);
-	}
-	else {
-		remove(TSAK_FIFO_FILE);
-	}
-	if (tsak) {
-		tsak->closeStdin();
-		tsak->closeStdout();
-		tsak->detach();
-		delete tsak;
-	}
 
 	if (trinity_desktop_synchronize_keyboard_lights &&
 	    TQString(getenv("DISPLAY")).startsWith(":")) {