aboutsummaryrefslogtreecommitdiff
path: root/slock.c
Commit message (Collapse)AuthorAgeFilesLines
* Remove cleanup and deglobalize and rework data structuresFRIGN2016-09-231-55/+23
| | | | | | | | | | | | | | | | | | | | | The cleanup removal is a joint-venture with Markus. We assume the X server does the cleanup, so we don't need it. The idea is that the fds are closed at exit and thus already indicate to the X server that the client has quit. Analogously the same applies to freeing memory sections previously allocated for the X server. We love XXXXXL burgers and therefore removed XUngrabPointer XUngrabKeyboard XFreeColors XFreePixmap XDestroyWindow Lines of Code. For a project like slock there is no need to carry around global state. By moving the three structures to main() it is now clear which functions modify which state, greatly improving the readability of the code, especially given slock is a suid program.
* Ensure Polyphemus-Mitigation and properly drop privilegesFRIGN2016-09-081-5/+25
| | | | | | | | | | | | | | | | | | | | | | | | | Don't hide privilege drops inside readpw() and actually make it configurable what you are dropping to in config.h. The privilege drop comes after opening the Display because the user "nobody" with "nogroup" can't do that. So why do I call this strategy the Polyphemus-Mitigation? """ After the giant returns in the evening and eats two more of the men, Odysseus offers Polyphemus some strong and undiluted wine given to him earlier on his journey. Drunk and unwary, the giant asks Odysseus his name, promising him a guest-gift if he answers. Odysseus tells him "Οὖτις", which means "nobody" and Polyphemus promises to eat this "Nobody" last of all. With that, he falls into a drunken sleep. Odysseus had meanwhile hardened a wooden stake in the fire and now drives it into Polyphemus' eye. When Polyphemus shouts for help from his fellow giants, saying that "Nobody" has hurt him, they think Polyphemus is being afflicted by divine power and recommend prayer as the answer. """ (source: https://en.wikipedia.org/wiki/Polyphemus)
* Unify how we check passwords between different OSesQuentin Rameau2016-09-071-34/+13
|
* make error message prefix consistentMarkus Teich2016-09-071-4/+4
|
* increasing for loops are idiomaticMarkus Teich2016-09-021-1/+1
|
* Localize running and failure inside readpw()Quentin Rameau2016-09-021-6/+4
| | | | They are only needed there, so don't make them global.
* Move screen unlocking inside cleanup()Quentin Rameau2016-09-021-9/+11
|
* Re-introduce the waiting loop for input grabbingQuentin Rameau2016-09-021-22/+38
| | | | | | | We actually “need” to wait a little for input to be released before locking for cases where slock is spawned from other graphical applications using keybindings. This undoes the misbehaviour I introduced in c2f9757, sorry for the mess.
* Add cleanup() to do free(locks) + XCloseDisplay()Quentin Rameau2016-09-021-6/+10
|
* Refactor dontkillme()FRIGN2016-08-311-7/+15
| | | | | | | | | | | | | | | - Use file pointers instead of raw I/O, inspired by Kernel code. - Use OOM_SCORE_ADJ_MIN from linux/oom.h instead of working with magic values. - Stricter error checking and descriptive error messages. The reasoning for using the constant rather than magic values lies in the fact that this ensures people get the message. With "-1000", a code reviewer would question if that is really the lowest possible number or just an arbitrary value. The kernel ABI probably won't change, but even in the case, we wouldn't have to modify the code. The OOM killer only is guaranteed to not kill you if you have OOM_SCORE_ADJ_MIN.
* Convert manpage to mandoc and fix usageFRIGN2016-08-311-1/+1
| | | | | | | | | In all honor, the previous usage was formally more correct, but for the sake of consistency across all the tools having the v-flag, I separated it from the command-string. Also, make use of the mandoc macros for the manpage. This makes it easier to maintain, extend and change in the future.
* fix CVE-2016-6866Markus Teich2016-08-311-2/+8
|
* Update bsd-auth string.Markus Teich2016-08-311-1/+1
| | | | Thanks to Hiltjo for discovering this.
* Exit as soon as possible on input grabbing errorQuentin Rameau2016-08-301-24/+20
| | | | | | We want to know at once if slock failed or not to lock the screen, not seing a black screen for a whole second (or two) and then die. Thanks to ^7heo for reporting this.
* Refactor main()FRIGN2016-08-221-31/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Add arg.h and fix usage Given slock is suid we don't want to have half-measures in place to parse the arguments in case the code is changed in the future with somebody not paying enough attention. Also, fix the usage string output to be more consistent across the suckless toolbase and make it reflect the manpage entry. - Comments Use proper block comments and add/change them where necessary to help in studying the code. - Error messages Consistently prepend them with "slock:" and fix wording and do a proper cleanup before quitting (XCloseDisplay and free the locks), making the die() semantics consistent with st's. - getpwuid() error reporting Properly present an error message if getpwuid() fails. - fork() error reporting Properly present an error message if fork() fails. If we cannot close the connection within the fork context we abort the operation and report an error. - execvp() error handling If execvp fails, we cannot call die() afterwards as this implies calling exit(). We must use _exit() to prevent the libc from doing now "illegal" cleanup-work.
* clear passwords with explicit_bzeroHiltjo Posthuma2016-08-131-2/+6
| | | | | | | | | | | | | | | | Make sure to explicitly clear memory that is used for password input. memset is often optimized out by the compiler. Brought to attention by the OpenBSD community, see: https://marc.info/?t=146989502600003&r=1&w=2 Thread subject: x11/slock: clear passwords with explicit_bzero Changes: - explicit_bzero.c import from libressl-portable. - Makefile: add COMPATSRC for compatibility src. - config.mk: add separate *BSD section in config.mk to simply uncomment it on these platforms.
* Revert "No need for usage()"Markus Teich2016-02-151-2/+15
| | | | | | | | | | This reverts most of commit a6dc051e3744ce5b14c54d2d246d3e8258207e76 and fixes some related stuff: - keep spelling fixes from original commit - make -h and -v also work when followed by more arguments - any unknown flag prints usage - fix output of -v to display "slock: version 1.3" instead of "slock: slock-1.3"
* revert using argv0 and minor fixupMarkus Teich2016-02-151-9/+6
| | | | | | - use hardcoded "slock" instead of argv[0] - add "slock: " to fprintf calls, where it was missing - revert `argc--, argv++` shifting
* Use argv0 instead of passing "slock:" to die every timeFRIGN2016-02-141-12/+16
|
* No need for usage()FRIGN2016-02-141-14/+2
| | | | | | | | | | | | | | | | | There are 2 arguments why -v and -h are broken: 1) if you are running off git, -v will show the last stable release, effectively making this option useless. people running stable versions leave open an attack surface this way in case there are vulnerabilities found. 99% of the people are also using package managers to keep their software up to date, instead of running $TOOL -v to check how old it is. 2) -h is a sad excuse for not just looking at the manual page (man 1 slock). Given we accept a post_lock_command, we can't be as liberal and just intercept certain flags. I changed the manpage to reflect this change.
* Simplify the oom-taming-functionFRIGN2016-02-141-14/+6
| | | | | | | | | | | | There really is no need to source a defined variable from a linux header. The OOM-rank ranges from -1000 to 1000, so we can safely hardcode -1000, which is a sane thing to do given slock is suid and we don't want to play around too much here anyway. On another notice, let's not forget that this still is a shitty heuristic. The OOM-killer still can kill us (thus I also changed the wording in the error-message. We do not disable the OOM-killer, we're just hiding.
* update copyright year in -v outputMarkus Teich2016-02-111-1/+1
|
* add hint for suid to oom error messageMarkus Teich2016-01-291-1/+1
|
* error messages on grab failureMarkus Teich2016-01-181-14/+16
|
* code style fixMarkus Teich2016-01-181-18/+9
|
* add option to run command after screen is lockedMarkus Teich2015-12-261-2/+10
|
* Don't change to failure colour on successDavid Phillips2015-08-271-1/+1
|
* Slightly safer OOM killer disablement in linuxDavid Phillips2015-07-061-1/+12
|
* rework setting window colorMarkus Teich2015-05-081-10/+7
|
* consistently use () with sizeofMarkus Teich2015-05-081-2/+2
|
* Option to not show failure color on clearNick Currier2015-05-081-3/+6
|
* resize lockscreen window after Xrandr resizeMarkus Teich2015-04-011-2/+16
|
* Blank the screen with color 0, add third color for failed loginsDavid Phillips2015-04-011-9/+18
| | | | | | | | | | | | | | | - Adds another color in config.def.h, COLOR_INIT - Renames the colours from numerical ones to ones with meaningful names; COLOR_INPUT for when there is content in the input buffer and COLOR_EMPTY for when the input buffer has been cleared (backspaced or a failed attempt). - Ensures XFreeColors frees the right number of colours. This is now derived from the size of `Lock->colors` rather than being an integer literal. - Makes slock exhibit the behaviour described by Markus The default colours are the same as the ones slock currently uses, with the exception of the new color, which I have set to red, as it indicates someone has either failed an attempt to unlock, or that they have entered input and erased it all.
* applied Dimitris' style patch from Dec'14, with some minor modificationsAnselm R Garbe2015-01-271-52/+60
|
* fixed usage string1.2Anselm R Garbe2014-12-221-1/+1
|
* applied sin's patch and prepared new releaseAnselm R Garbe2014-12-221-0/+2
|
* end{pw,sp}ent() can only be called after get{pw,sp}ent()sin2014-11-091-2/+0
| | | | Calling them unconditionally can result in memory corruption.
* Only check errno if getpwuid() failssin2014-07-091-4/+6
| | | | Checking errno otherwise is unspecified.
* Add /etc/passwd supportFRIGN2014-06-091-2/+2
| | | | | Fix slock to work with /etc/passwd without /etc/shadow. while we're at it, remove an occurence of trailing whitespace.
* Set errno to 0 before getpwuid() and check it afterwardssin2014-01-291-1/+4
|
* applied Robert Schneider's Linux suggestions, also bumped version and ↵Anselm R Garbe2013-08-021-8/+28
| | | | updated LICENSE file's copyright notice
* applied andres' multi-slock fix, thanks for spotting this issuegarbeam@gmail.com2012-08-021-3/+14
|
* added Ben's password placeholder entry kludgeanselm@garbe.us2012-04-151-1/+1
|
* applied Eckehard Bern's dualcolor patch to slockanselm@garbe.us2012-03-171-5/+22
|
* select for SubstructureRedirectMask as wellanselm@garbe.us2012-02-091-0/+2
|
* fixed missing new linesanselm@garbe.us2012-02-091-2/+2
|
* typo fixanselm@garbe.us2012-02-051-1/+1
|
* added XRaiseWindow workaround when new clients are launchedanselm@garbe.us2012-02-051-51/+35
|
* applied Michaels patchgarbeam@gmail.com2012-01-241-2/+4
|
* added dummy error handler to prevent X from terminating slock just in caseanselm@garbe.us2012-01-221-8/+14
|
generated by cgit v1.2.3-70-g09d2 (git 2.47.0) at 2024-11-29 03:28:20 +0000