Reformat according to current Opie style.

1 files changed, 25 insertions, 23 deletions

diff --git a/core/opie-login/main.cpp b/core/opie-login/main.cpp
index 3f1077c..b76d2c7 100644
--- a/core/opie-login/main.cpp
+++ b/core/opie-login/main.cpp
@@ -72,14 +72,17 @@ int main ( int argc, char **argv )
                 ::fprintf ( stderr, "%s can only be executed by root. (or chmod +s)\n", argv [0] );
                 return 1;
         }
-        if ( ::getuid ( ) != 0 ) // qt doesn't really like SUID and
-                ::setuid ( 0 );      // messes up things like config files
-
-        //struct rlimit rl;
-        //::getrlimit ( RLIMIT_NOFILE, &rl );
-
-        //for ( unsigned int i = 0; i < rl. rlim_cur; i++ )
-        //      ::close ( i );
+    /*!
+     * @bug
+     * Qte does not really like being set UID root. This is
+     * largely because we do almost everything on config files
+     * in root context. So if you even want to use opie-login
+     * you are in for a world of hurt unless someone at least
+     * scrubs the settings area and the PIM apps to make sure that
+     * they are covered regarding perms and users.
+     */
+    if ( ::getuid ( ) != 0 )
+        ::setuid ( 0 );
 
         ::setpgid ( 0, 0 );
         ::setsid ( );
@@ -94,7 +97,8 @@ int main ( int argc, char **argv )
         Config c( "opie-login" );
         c.setGroup( "autologin" );
         QString entry = c.readEntry( "user", "" );
-        if ( !entry.isEmpty() ) autolog = ::strdup( (const char*) entry );
+    if ( !entry.isEmpty() )
+        autolog = ::strdup( (const char*) entry );
         
         while ( true ) {
                 pid_t child = ::fork ( );
@@ -107,7 +111,8 @@ int main ( int argc, char **argv )
                         int status = 0;
                         time_t started = ::time ( 0 );
 
-                        while ( ::waitpid ( child, &status, 0 ) < 0 ) { }
+            while ( ::waitpid ( child, &status, 0 ) < 0 )
+                ;
 
                         LoginApplication::logout ( );
 
@@ -180,13 +185,11 @@ int main ( int argc, char **argv )
                         }
 
                         if ( autolog && !userExited ) {
-
                 QWSServer::setDesktopBackground( QImage() );
                 ODevice::inst()->setDisplayStatus( true );
                 LoginApplication *app = new LoginApplication ( argc, argv, ppid );
                 LoginApplication::setLoginAs( autolog );
 
-
                                 if ( LoginApplication::changeIdentity ( ))
                                         ::exit ( LoginApplication::login ( ));
                                 else
@@ -245,6 +248,7 @@ public:
         return true;
         break;
       case 2:
+                default:
         // We're going to suspend the whole machine
         if ( PowerStatusManager::readStatus().acStatus() != PowerStatus::Online ) {
           QWSServer::sendKeyEvent( 0xffff, Qt::Key_F34, FALSE, TRUE, FALSE );
@@ -255,8 +259,6 @@ public:
     return false;
   }
 
-private:
-public:
   void setIntervals( int i1 = 30, int i2 = 20, int i3 = 60 )
   {
     int v [4];
@@ -289,6 +291,7 @@ public:
     }
     if ( m_backlight_forcedoff && bright != -2 )
       return ;
+
     if ( bright == -2 ) {
       // Toggle between off and on
       bright = m_backlight_bright ? 0 : -1;
@@ -305,7 +308,6 @@ public:
 
 private:
   bool m_lcd_status;
-
   int m_backlight_bright;
   bool m_backlight_forcedoff;
 };
@@ -320,7 +322,8 @@ int login_main ( int argc, char **argv, pid_t ppid )
         app-> setStyle ( new QPEStyle ( ));
 
         if ( QWSServer::mouseHandler() &&
-             QWSServer::mouseHandler() ->inherits("QCalibratedMouseHandler") ) {
+         QWSServer::mouseHandler()-> inherits("QCalibratedMouseHandler") )
+    {
                 if ( !QFile::exists ( "/etc/pointercal" )) {
                         // Make sure calibration widget starts on top.
                         Calibrate *cal = new Calibrate;
@@ -335,10 +338,10 @@ int login_main ( int argc, char **argv, pid_t ppid )
         QWSServer::setScreenSaver ( saver );
         saver-> restore ( );
 
-
         LoginWindowImpl *lw = new LoginWindowImpl ( );
         app-> setMainWidget ( lw );
-        lw-> setGeometry ( 0, 0, app-> desktop ( )-> width ( ), app-> desktop ( )-> height ( ));
+    lw-> setGeometry ( 0, 0, app-> desktop ( )-> width ( ),
+                       app-> desktop ( )-> height ( ));
         lw-> show ( );
 
         int rc = app-> exec ( );
@@ -346,17 +349,16 @@ int login_main ( int argc, char **argv, pid_t ppid )
         if ( app-> loginAs ( )) {
                 if ( app-> changeIdentity ( )) {
                         app-> login ( );
-
                         // if login succeeds, it never comes back
-
-                        QMessageBox::critical ( 0, LoginWindowImpl::tr( "Failure" ), LoginWindowImpl::tr( "Could not start Opie." ));
+            QMessageBox::critical ( 0, LoginWindowImpl::tr( "Failure" ),
+                                    LoginWindowImpl::tr( "Could not start Opie." ));
                         rc = 1;
                 }
                 else {
-                        QMessageBox::critical ( 0, LoginWindowImpl::tr( "Failure" ), LoginWindowImpl::tr( "Could not switch to new user identity" ));
+            QMessageBox::critical ( 0, LoginWindowImpl::tr( "Failure" ),
+                                    LoginWindowImpl::tr( "Could not switch to new user identity" ));
                         rc = 2;
                 }
-
         }
         return rc;
 }