summary refs log tree commit diff
path: root/util.c
Commit message (Collapse)AuthorAgeFilesLines
* Prevent out of boundary write with configuration files in which too manytobias2020-04-161-1/+1
| | | | | | | | | | quoted arguments are stored for other window managers. The quotation handling happens within the while loop without checking if the "end" limit has been already reached. If this happens, the final NULL assignment leads to an out of boundary write on stack. OK okan@
* Unrelated style fixes, consistency changes and sorting, appropriateokan2020-02-271-3/+3
| | | | | dosage/removal of wrappers, simplification of name queue, client cycle joins other kb/mb bound functions.
* Plug a memory leak in log_debug(); OK okan@tim2019-04-291-0/+1
|
* fix missing includesokan2019-02-131-0/+1
|
* Slightly expand and expose verbose debugging.okan2018-02-041-3/+5
|
* Add a simple debug logging mechanism.okan2018-02-021-0/+30
|
* Quick fix: exit after a failed execvp in u_spawn instead; previously we did inokan2018-01-081-1/+1
| | | | | | | | u_exec, but the introduction of re-exec'ing the previous invocation of cwm if 'exec_wm' failed missed the 'exec' failing path. Will likely split out as a proper fix. Odd behaviour reported by Ve Telko.
* If the replacement window manager fails to start, restart the fallback (theokan2017-12-291-1/+1
|\ | | | | | | original invocation of cwm).
* | On execwm, we should properly release resources before exec'ing into aokan2015-09-161-4/+28
| | | | | | | | | | | | | | new window manager; so allow CWM_EXEC_WM to assign new wm to wm_argv and pass through cwm_status (now EXECWM) so that x_teardown() gets called before exec'ing the new window manager. Removes the need for a separate x_restart() now, using new wm_argv; and consolidates errno for execvp.
* | Switch to limits.h; replace MAXPATHLEN and MAXHOSTNAMELEN with PATH_MAXokan2015-01-191-1/+2
| | | | | | | | | | | | and HOST_NAME_MAX+1, respectively. ok doug@
* | sortokan2012-11-091-1/+1
| |
* | We are inconsistent when it comes to function returns, so just go allokan2011-07-251-2/+2
| | | | | | | | | | | | the way with the cwm specific parts. ok oga@
* | alter a few function returns and prototypes; found by lint.okan2011-06-241-4/+1
| | | | | | | | ok oga@
* | tag and comment cleanup; ok oga@okan2011-05-111-1/+1
| |
* | pull all non-X11 headers from calmwm.h and place them only where theyokan2009-12-151-0/+10
| | | | | | | | | | | | are required. encourged to go all the way by oga@
* | merge the 2 common header files; specific includes to be pulled out asokan2009-12-151-1/+0
| | | | | | | | | | | | separate commits. ok oga@
* | spacing, declaration lineup to be consistent throughout cwm,okan2008-07-111-3/+2
| | | | | | | | | | | | readability, and a bit of knf. ok oga@
* | KNF, no binary change.oga2008-05-151-2/+2
| | | | | | | | From Pierre Riteau. Thanks!
* | make the argument parser for commands accept quoted strings, while i'moga2008-04-151-15/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | there make u_spawn use exec_wm (renamed to u_exec) for it's execution to remove duplicated code. This means constructs like this work in .cwmrc: bind CM-t "ssh -Y 192.168.1.2 \"xterm -e top\"" or alternatively: bind CM-t "ssh -Y 192.168.1.2 'xterm -e top'" "in it goes" okan@.
* | Kill dirent_isdir() and dirent_islink() nothing used them since the newoga2008-04-151-28/+0
| | | | | | | | | | | | parser went in. ok okan.
* | hit it with the knf stick.oga2008-04-151-18/+20
| |
* | add exec_wm to calmwm.h, it was missing (bad ian@!)oga2008-02-201-1/+1
| | | | | | | | | | | | | | | | | | Also, exec_wm() currenty err()s out if something failed with the exec, killing the currently running wm. This is bad. Replace the err() with warn() instead. from Gleydson Soares, thanks!
* | huge amount of cleanup and dead code removal.oga2008-01-161-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | full description of changes: -remove fontlist, and all associated structures/calls, it's not needed. this also removes any doubt about leftover 9wm code (the list was borrowed from it). Since cwm now uses Xft for everything, the legacy font handling is just not needed. -add /* FALLTHROUGH */ comments into grab_{label,menu}. I actually didn't intend grab_menu to be a fallthrough, but it actually works quite well there, so remove the extra rectangle drawing. I love it when that happens. -remove a couple of unused prototypes that were obviously missed before. -remove a bunch of commented out or if 0ed out code. It doesn't look to be coming back anytime soon. -several functions returned an int, but this was never checked. most of them only failed if they failed to grab the pointer (thus the internal state didn't change), so just make them void and return early if this is the case. -remove several unused functions and some useless variables. knocks something like 200bytes off the stripped binary size for me. ok marc@, tested by several others.
* | - Remove the "all rights reserved" tag at the top of most of the sourceoga2008-01-111-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | files, and replace them with the actual ISC license. - add license to the manpage (it was lacking before) - correct license statement in the README Permission given by Marius (copyright holder): "1. please replace with the standard ISC license 2. you may add the ISC license to the man page 3. feel free to replace the information in the README as well" and Dros (copyright holder for group.c): "Please switch group.c to the ISC License." ok ian@
* | Add a "restart wm" function. ok oga@ian2007-11-281-0/+16
|/
* Initial revisionbernd2007-04-271-0/+72