On Thu, Jan 6, 2011 at 1:58 AM, Enlightenment SVN <no-re...@enlightenment.org> wrote: > Log: > desklock - dont ret 1 on failure to auth. only on failure to init! > > > > Author: raster > Date: 2011-01-05 19:58:58 -0800 (Wed, 05 Jan 2011) > New Revision: 55923 > Trac: http://trac.enlightenment.org/e/changeset/55923 > > Modified: > trunk/e/src/bin/e_desklock.c > > Modified: trunk/e/src/bin/e_desklock.c > =================================================================== > --- trunk/e/src/bin/e_desklock.c 2011-01-06 03:57:37 UTC (rev 55922) > +++ trunk/e/src/bin/e_desklock.c 2011-01-06 03:58:58 UTC (rev 55923) > @@ -718,7 +718,7 @@ > if (pamerr != PAM_SUCCESS) > { > free(current_user); > - exit(pamerr); > + exit(1); > } > pamerr = pam_authenticate(da.pam.handle, 0); > pam_end(da.pam.handle, pamerr); > @@ -730,7 +730,7 @@ > exit(0); > } > free(current_user); > - exit(1); > + exit(-1);
Why? It's wrong to exit with a negative value. Lucas De Marchi ------------------------------------------------------------------------------ Gaining the trust of online customers is vital for the success of any company that requires sensitive data to be transmitted over the Web. Learn how to best implement a security strategy that keeps consumers' information secure and instills the confidence they need to proceed with transactions. http://p.sf.net/sfu/oracle-sfdevnl _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
