A couple more return values that need to be checked.

1 files changed, 5 insertions, 1 deletions

diff --git a/core/opie-login/passworddialogimpl.cpp b/core/opie-login/passworddialogimpl.cpp
index d9132e2..3c1b474 100644
--- a/core/opie-login/passworddialogimpl.cpp
+++ b/core/opie-login/passworddialogimpl.cpp
@@ -71,167 +71,171 @@ static char *crypt_make_salt() {
     static unsigned long x;
     static char result[3];
 
     ::time(&now);
     x += now + getpid() + clock();
     result[0] = i64c(((x >> 18) ^ (x >> 6)) & 077);
     result[1] = i64c(((x >> 12) ^ x) & 077);
     result[2] = '\0';
     return result;
 }
 
 /*
  * Modal dialog to force root password. It is quite hard as it only leave
  * when a password is set.
  * Also it prevalidates the password...
  */
 PasswordDialogImpl::PasswordDialogImpl( QWidget* parent )
     : PasswordDialog( parent, 0, true ), m_isSet( PasswordDialogImpl::needDialog() ) {
 }
 
 PasswordDialogImpl::~PasswordDialogImpl() {
     /* qt does the stuff for us */
 }
 
 void PasswordDialogImpl::done(int res) {
     m_isSet = true;
 
     /*
      * The user hit 'Ok' see if we can safe the file
      * if not an error will be raised and m_isSet altered.
      * On cancel we will see if it is now ok...
      */
     if ( res == Accepted )
         writePassword();
     else if(PasswordDialogImpl::needDialog() ) {
         switch( QMessageBox::warning(this,tr("Trying to leave without password set") ,
                                      tr("<qt>No password was set. This could lead to you not beeing"
                                         "able to remotely connect to your machine."
                                         "Do you want to continue not setting a password?</qt>" ),
                                      QMessageBox::Ok, QMessageBox::Cancel ) ) {
         case QMessageBox::Cancel:
             m_isSet = false;
             break;
         case  QMessageBox::Ok:
         default:
             break;
         }
 
     }
 
     if(m_isSet)
         PasswordDialog::done( res );
 }
 
 /*
  * Lets see if we can write either shadow
  *
  */
 /**
  * CRYPT the password and then tries to write it either to the shadow password
  * or  to the plain /etc/passwd
  */
 void PasswordDialogImpl::writePassword() {
     /*
      * Check if both texts are the same
      */
     if ( m_pass->text() != m_confirm->text() )
         return error( tr("Passwords don't match"),
                       tr("<qt>The two passwords don't match. Please try again.</qt>") );
 
 
     /*
      * Now crypt the password so we can write it later
      */
     char* password = ::crypt( m_pass->text().latin1(), crypt_make_salt() );
 
     if ( !password )
         return error( tr("Password not legal" ),
                       tr("<qt>The entered password is not a valid password."
                          "Please try entering a valid password.</qt>" ) );
 
     /* rewind and rewrite the password file */
     ::setpwent();
 
     FILE* file = ::fopen( "/etc/passwd.new", "w" );
     struct passwd* pass;
     while ( (pass = ::getpwent()) != 0l ) {
         /* no  shadow password support */
         if ( pass->pw_uid == 0  )
             pass->pw_passwd = password;
 
         ::putpwent( pass, file );
     }
 
     ::fclose( file );
     ::endpwent();
-    ::rename("/etc/passwd.new","/etc/passwd" );
+    if (::rename("/etc/passwd.new","/etc/passwd" ) == -1)
+        return error( tr("Rename /etc/passwd failed"),
+                      tr("<qt>Renaming /etc/passwd.new to /etc/passwd failed."
+                         "Please check your /etc/passed file, your /etc directory "
+                         "or your filesystem.</qt>") );
 
     /* should be done now */
 #ifdef  OPIE_LOGIN_SHADOW_PW
     #error "Can't write Shadow Passwords fixme"
 #endif
 }
 
 /**
  * Raise an error. Delete input and set the focus after showing
  * the error to the user
  */
 void PasswordDialogImpl::error( const QString& caption, const QString& text ) {
     m_isSet = false;
     QMessageBox::critical(this,caption, text,
                           QMessageBox::Ok, QMessageBox::NoButton );
 
     m_pass->setText("");
     m_pass->setFocus();
 
     m_confirm->setText("");
 }
 
 void PasswordDialogImpl::slotToggleEcho( bool b ) {
     m_pass->   setEchoMode( b ? QLineEdit::Normal : QLineEdit::Password );
     m_confirm->setEchoMode( b ? QLineEdit::Normal : QLineEdit::Password );
 }
 
 /////////////////////////
 /// static functions
 ///
 
 /**
  * Ask whether or not we need to show the dialog. It returns true if
  * no root password is set so that the user will be able to set one.
  */
 bool PasswordDialogImpl::needDialog() {
    /*
     * This can cope with no password and shadow passwords
     * Let us go through the user database until we find  'root' and then
     * see if it is 'shadow' and see if shadow is empty or see if the password is empty
     */
     bool need = false;
     struct passwd *pwd;
     ::setpwent();
 
     while((pwd = ::getpwent() ) ) {
         /* found root */
         if( pwd->pw_uid == 0 ) {
             QString str = QString::fromLatin1(pwd->pw_passwd );
 
             /*
              * If str is really empty it is passwordless anyway... or '*' is a hint to set one
              * on OE/Familiar
              * else it is shadow based
              */
             if(str.isEmpty() || str == '*' )
                 need = true;
             else if ( str == 'x' )
 #ifdef OPIE_LOGIN_SHADOW_PW
                 need =  QString::fromLatin1( ::getspnam( pwd->pw_name )->sp_pwdp ).isEmpty();
 #else
             ;
 #endif
             break;
         }
     }
     ::endpwent();
 
     return need;
 }