README: refer to cr.i3wm.org for patches

Limit the debug print to the actual password size

Signed-off-by: Quentin Glidic <sardemff7+git@sardemff7.net>

RGB color notation is 3-byte hexadecimal

Signed-off-by: Vivien Didelot <vivien.didelot@gmail.com>

Allow passwordless PAM conversations (e.g. fingerprint)

Allow XKB_KEY_XF86ScreenSaver as synonm for enter

This keysym is generated on convertible tablets by pressing a hardware
lock/unlock button.

remove support for NOLIBCAIRO, cairo-xcb is widespread by now

README: remove unhelpful comment about pkg-config

use libxkbcommon for input handling

Thanks to Ran Benita and Daniel Stone (the libxkbcommon authors) for
answering my questions and reviewing this code.

With this commit, input handling should be more correct with using less
code (in i3lock, that is).

bugfix: free(reply) to avoid memory leak

s/pam_message/pam_response/ (Thanks Tucos)

The previous code didn’t cause any issues since both types (currently!)
have the same size.

promote the "could not load image" message from debug to normal

only use mlock() on Linux, FreeBSD (for example) requires root

getopt_long returns int, don't cast to char.

On ARM, casting -1 to char turns it into something else. Fixes
argument parsing on ARM. Thanks Chipaca for reporting.

update changelog for 2.4.1

skip shift when getting the modifier mask (Thanks SardemFF7)

revert shift lock handling (broke uppercase letters)

With some layouts, this broke uppercase letters in your passwords.

I think that explicit shiftlock handling is unnecessary. X11 seems to do
it on its own. Here is what leads me to that conclusion:

  $ setxkbmap de
  $ xmodmap -e 'keycode 66 = Shift_Lock'
  $ xev

Now enter a character, say "a", then press CapsLk (which is now
Shift_Lock), then press "a" again. The event state is 0x1, thereby
undistinguishable from normal shift.

Bugfix: Correctly center unlock indicator after reconfiguring screens (Thanks xn)

Fixes: #712

remove debian/

The Debian packaging is now hosted on Debian servers at
git://git.debian.org/users/stapelberg/i3lock.git

This makes life easier for Debian people. It makes life harder
for you if you want to build a Debian package of the current
git version. Here is how you could do that now:

Build a tarball of the current git version:
mkdir ../i3lock-dpkg
VERSION=i3lock-$(git describe --tags)
git archive --prefix=$VERSION/ --output=../i3lock-dpkg/$VERSION.tar.bz2 HEAD

get the packaging:
cd ../i3lock-dpkg
gbp-clone git://git.debian.org/users/stapelberg/i3lock.git
cd i3lock
git-import-orig ../$VERSION.tar.bz2
dpkg-buildpackage

add .gitignore

add CHANGELOG file

Use (void) instead of () for functions without args (Thanks fernandotcl)

See also:
http://article.gmane.org/gmane.linux.kernel/1268792

The C compiler will handle (void) as "no arguments" and () as "variadic
function" (equivalent to (...)) which might lead to subtle errors, such
as the one which was fixed with commit 0ea64ae4.

Bugfix: fallback when the image cannot be loaded

If the specified file does not exist or is invalid, previously, the unlock
indicator wouldn’t show up at all, because the invalid surface was still used.

With this commit, i3lock will react like if you didn’t specify an image at all.

Only output text when in debug mode (fixes problems with xautolock)

Bugfix: Fix background color when using cairo (Thanks Pascal)

debian: update changelog for 2.3.1

fix compilation on some systems (Thanks badboy)

debian: fix packaging format to 1.0 for now

debian: remove duplicate priority field

manpage: add -u flag in synopsis

manpage: hyphen

debian: update packaging

README: also add libev to dependencies

document the dependency on libxcb-xinerama

Properly handle Caps Lock (Thanks Damien)

man page: mention xautolock

Bugfix: Copy the color depth from parent (root) window instead of hardcoding to 24 (Thanks hafron)

Added missing #ifndef NOLIBCAIRO

Needed to compile without libcairo.

Bugfix: Check for waiting X11 events before entering the event loop

Change the unlock indicator properly after pressing the 'escape' key

debian: add libxcb-xinerama0-dev as build-dependency

draw the unlock indicator in the middle of every screen

Implement Xinerama support (not used yet)

Fix a few memory leaks

Properly free timeouts, move one of the timeouts to unlock_indicator.c

remove the dead variable modeswitchmask

Properly free the reply of xcb_get_geometry

Refactor the unlock indicator into its own source code file

Fork after the window is visible, not before

change ALL the copyrights!

Lock the password buffer in memory, clear it in RAM after verifying

support ISO_Level5_Shift (Thanks roland)

Fix the timeouts for hiding the unlock indicator (Thanks Merovius)

Properly free X11 resources (fixes DoS bug) (Thanks Merovius)

Implement a visual unlock indicator

debian: update packaging to use dpkg-buildflags, i3lock.manpages

Makefile: add 'dist' target

Properly implement -h (fixes error message about unknown option)

Get rid of superfluous \n in --version

update debian/changelog

Bugfix: Properly detect the release of ISO_Level3_Shift (Thanks bacardi55)

Introduce --debug and some debugging options related to the Mode_switch / ISO_Level3_Shift thing

Correctly handle Mode_switch/ISO_Level3_Shift (Thanks bacardi55)

I tested this with the following experiment:
$ setxkbmap 'us(intl)'
$ xmodmap ~/configfiles/midna/Xmodmap
$ xmodmap -e 'keycode 38 = a A adiaeresis Adiaeresis o O'
$ xmodmap -e 'keycode 49 = ISO_Level3_Shift ISO_Level3_Shift ISO_Level3_Shift ISO_Level3_Shift'
Then, Mode_switch + a yields ä, but ` + a yields o.

In i3lock, these were swapped (Mode_switch + a yielded o, while ä was not
reachable at all). The comment in the code explains it

(See http://code.stapelberg.de/git/configfiles for the Xmodmap)

Bugfix: Don’t re-grab pointer/keyboard on MappingNotify (Thanks wilmer)

This fixes bug http://bugs.i3wm.org/545, where characters (of your password)
would rarely slip through when entering your password (especially) after
resuming your notebook from suspend to RAM.

The reason is that when resuming, X triggers one or more MappingNotify events.
At the same time, CPU load is high. This leads to a race-condition between the
ungrab and re-grab in which i3lock temporarily does not grab the keyboard.

One way to fix this is using xcb_grab_server() before and xcb_ungrab_server()
after the ungrab/re-grab. However, I think we actually don’t need to
ungrab/re-grab at all. I seem to have put that code in here by mistake – in i3,
we re-grab after MappingNotify, but there we only grab specific keys. In
i3lock, we grab the whole pointer/keyboard, so there should be no need.

If I’m incorrect and this breaks some subtle use-cases for people with strange
layout setup, at least we can properly document on why we need it, after we put
it back in ;).

manpage: mention arguments in OPTIONS section, too (Thanks TrickSTer)

Throw an error when specifying an invalid pointer argument (-p) (Thanks TrickSTer)

Fixed handle_screen_resize to build with NOLIBCAIRO enabled

handle screen resolution images and redraw the lock window

update debian packaging, update copyright

makefile: respect and use CPPFLAGS / LIBS (Thanks Kacper)

optimization: render to pixmap which is used as background for the window

Before this commit, the background color (white by default) was visible for
about 100ms until the image was drawn. This flickering is now eliminated.

Also, we don’t need to handle Expose-events anymore, as X11 will use the
window’s background pixmap automatically.

Mark handle_visibility_notify and handle_mapping_notify as static

update debian packaging

cleanup indention left-overs

bump copyright

Handle MappingNotify-events (Thanks Pascal)

use 'auth include login' instead of '@include login', the latter is debian-specific (Thanks mseed)

add i3lock.pam (installed as /etc/pam.d/i3lock) which includes 'login' (Thanks Семён)

Bugfix: correctly check for errors when connecting to X11

add XK_KP_Enter as synonym for XK_Return

Bugfix: Handle numpad keys correctly (Thanks Pascal)

Bugfix: When entering 512 characters, you could use neither return nor backspace (Thanks carl)

Add a hint to the error message 'unknown option' when compiled with NOLIBCAIRO

Make it possible to build i3lock without cairo.

comment out debug output (Thanks Ciprian)

debian: change priority to optional

update debian packaging for i3lock 2.0 release

print version number when invoked with -v

manpage: add missing "specify" (Thanks xeen)

s/passwords/password (Thanks xeen)

Re-implement tiling of image (-t)

Fix compilation warning by including cairo-xcb

Move static byte arrays to xcb.c (fixes compilation warnings)

Port the pointer-code to xcb

disable debug output which reveals the password

(full debug output will be disabled for the next release)

grabbing: make the retry loop much slower (waits up to half a second)

This should fix a race condition where the Mod-key was not released
in time when starting i3lock using a key combination.

Bugfix: Correctly error out when unable to grab pointer/keyboard

Raise the i3lock window upon visibility notifies

make event loop simpler (Thanks to Jamey Sharp)

As explained in Jamey’s post on the XCB mailing list, the event
loop can/should be written in a more simple way than it was before
in xcb-event. See:
<AANLkTinhIM320wUjSeHkBTVk4ysBuiSZkvfs8pMLVykO@mail.gmail.com>
or
http://lists.freedesktop.org/archives/xcb/2010-July/006292.html

err, xcb_image is not actually used

s/arry/array