Bastien Nocera
7a72d8fd58
lib: Fix uninitialised variable in fpi_imgdev_image_captured()
...
libfprint/fpi-dev-img.c:255:6: warning: variable 'print' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
if (imgdev->action != IMG_ACTION_CAPTURE) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
libfprint/fpi-dev-img.c:271:25: note: uninitialized use occurs here
imgdev->acquire_data = print;
^~~~~
libfprint/fpi-dev-img.c:255:2: note: remove the 'if' if its condition is always true
if (imgdev->action != IMG_ACTION_CAPTURE) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
libfprint/fpi-dev-img.c:232:29: note: initialize the variable 'print' to silence this warning
struct fp_print_data *print;
^
= NULL
2018-11-13 11:54:41 +01:00
Bastien Nocera
5edfd55e00
doc: Fix typo in chapter title
2018-11-12 12:10:11 +01:00
Bastien Nocera
2951daaa01
doc: Split driver docs into chapters
2018-11-09 15:32:39 +01:00
Bastien Nocera
23f7c4aaf7
doc: Hide G_LOG_DOMAIN from gtk-doc
...
It's only used internally, and doesn't need to be documented here,
as FP_COMPONENT which uses it already is.
2018-11-09 15:32:39 +01:00
Bastien Nocera
3dc5c4a6de
doc: Add LIBFPRINT_DEPRECATED to the docs
2018-11-09 15:32:39 +01:00
Bastien Nocera
a97ae3bc35
doc: Add fpi-async section to the docs
2018-11-09 15:32:39 +01:00
Bastien Nocera
53d2fb3ad2
doc: Add more functions to fpi-dev section
...
And rename section to match that of fpi-dev-img
2018-11-09 15:32:39 +01:00
Bastien Nocera
4f7e507716
doc: Add fpi-dev-img section to the docs
2018-11-09 15:32:39 +01:00
Bastien Nocera
f2e3a840db
doc: Add fpi-core and fpi-core-img sections to the docs
2018-11-09 15:32:39 +01:00
Bastien Nocera
f45c18116a
doc: Add fpi-data section to the docs
2018-11-09 15:32:39 +01:00
Bastien Nocera
2365c608f9
lib: Change "unsigned" to "unsigned int"
...
To appease gtk-doc's struct parser, which chokes on the short name for
the data type.
See https://gitlab.gnome.org/GNOME/gtk-doc/issues/63
2018-11-09 14:20:19 +01:00
Bastien Nocera
6c6daaa619
doc: Add fpi-assembling section to the docs
2018-11-09 14:08:04 +01:00
Bastien Nocera
857a399bfa
doc: Fix parameter names in fp_minutia_get_coords() docs
...
The API docs for fp_minutia_get_coords() as added in commit 1006467
had the parameters in the docs not match the actual names used.
2018-11-09 13:58:21 +01:00
Bastien Nocera
6e230f0a07
doc: Add missing fpi_timeout_set_name() to the docs
2018-11-09 13:58:21 +01:00
Bastien Nocera
7402401057
doc: Add fpi-img section to the docs
2018-11-09 13:58:21 +01:00
Bastien Nocera
fe17dfe01d
lib: Rename fpi_im_resize() to fpi_img_resize()
...
Now that this name is free. This makes all the image manipulation
functions have the same "fpi_img_" prefix.
2018-11-09 13:36:52 +01:00
Bastien Nocera
4885b38be5
lib: Rename fpi_img_resize() to fpi_img_realloc()
...
To better match what it does. It does not resize an image, but
reallocate its internal data structure's size.
2018-11-09 13:35:40 +01:00
Bastien Nocera
5d4a5981d0
lib: Make fpi_poll_is_setup() private
...
As it's not used in other parts of the library after the changes in
commit 422f81b
.
2018-11-09 13:33:02 +01:00
Bastien Nocera
0fcb4533b5
doc: Add more headers to ignore list
...
Those are not public headers, so don't need to be documented. This
removes 18 symbols from the undocumented symbols list.
2018-11-09 13:29:56 +01:00
Bastien Nocera
422f81b60c
Revert "lib: Async functions cannot be started without a mainloop"
...
This reverts commit 700c5791f8
.
We don't need a poll setup to be able to call sync functions, which then
use async functions for implementation internally.
Closes : #124
Conflicts:
libfprint/fpi-async.c
2018-11-09 12:22:45 +01:00
Bastien Nocera
e0d2f3ae53
lib: Link to fp_enroll_finger_img() in fp_enroll_stage_cb() doc
2018-10-23 11:24:56 +02:00
Bastien Nocera
531d9048b3
lib: Fix up third-person usage in API docs
2018-10-23 11:24:56 +02:00
Bastien Nocera
18495d122d
lib: Fix up @dev argument docs
2018-10-23 11:24:56 +02:00
Bastien Nocera
9bcacd97df
lib: Require a callback for a number of async calls
...
Otherwise the caller won't be able to call the appropriate _finish()
calls when done.
See https://gitlab.freedesktop.org/libfprint/libfprint/issues/119 for
the long-term plan
2018-10-23 11:24:56 +02:00
Bastien Nocera
d91eae26e5
lib: Add API docs for the async functions
2018-10-23 11:24:56 +02:00
Bastien Nocera
d113ed9d83
docs: Update API docs style
...
No period at the end of parameter docs.
2018-10-23 11:24:56 +02:00
Bastien Nocera
e113754312
docs: Add "struct #fp_dev" to the API docs
2018-10-23 11:24:56 +02:00
Bastien Nocera
b1afa9da5d
lib: No mainloop, no timeouts either
2018-10-23 10:59:15 +02:00
Bastien Nocera
700c5791f8
lib: Async functions cannot be started without a mainloop
...
No mainloop, no async.
2018-10-23 10:59:15 +02:00
Bastien Nocera
97d0a6dfe6
lib: Add fpi_poll_is_setup() sanity check
...
This checks whether polling was correctly setup for integration with a
mainloop.
2018-10-23 10:59:15 +02:00
Bastien Nocera
a54d020c97
lib: Make fp_get_pollfds() fail when fp_init() wasn't called
...
So things can't be called out-of-order.
2018-10-23 10:59:15 +02:00
Bastien Nocera
36b696f433
lib: Fix error messages in async capture functions
...
They mentioned verification instead of capture.
2018-10-23 10:59:15 +02:00
Bastien Nocera
f42cd6eefd
lib: Add fp_driver_supports_imaging()
...
So we don't need to open a device before checking whether it supports
imaging.
2018-10-23 10:59:15 +02:00
Bastien Nocera
46ebb39f65
lib: Throw an error if fp_init() wasn't called
...
You can't call fp_discover_devs() without calling fp_init() and having
it not fail, otherwise there's nothing to discover...
2018-10-23 10:59:15 +02:00
Bastien Nocera
c88a51cce8
build: Refuse to build without any drivers
2018-10-23 10:59:15 +02:00
Bastien Nocera
3a7b03f022
lib: Fix crash when too many minutiae were detected
...
struct xyt_struct uses a fixed-sized array to fit MAX_BOZORTH_MINUTIAE (200)
minutiae. MAX_FILE_MINUTIAE is 1000. So if we detected more than
MAX_BOZORTH_MINUTIAE, we would crash copying the data from the capture
to the structure.
We might want to use dynamically allocated arrays in the future (or
bigger ones) so that we don't lose minutiae.
Closes : #116
2018-10-12 12:27:37 +02:00
Bastien Nocera
94450a1d74
lib: Fix fpi_img_is_sane()
...
The checks weren't:
- checking whether the width or height were negative
- whether img->width * img->height would overflow, or
was bigger than G_MAXINT
- whether img->width * img->height was bigger than the total
length of the buffer
The last one looks like a thinko, it checked for:
(img->length * img->height) < img->length
which is equivalent to:
img->height < 1
which we already check for earlier.
Closes : #85
2018-10-12 00:54:39 +00:00
Bastien Nocera
c35ad20249
mindtct: Fix a memory leak in morph_TF_map()
...
cimage is leaked when mimage fails to allocate.
Spotted by Seth Arnold
Closes : #82
2018-10-12 00:54:39 +00:00
Bastien Nocera
5e8b4a81e9
mindtct: Fix leak in pixelize_map()
...
pmap was not freed in error cases.
Spotted by Seth Arnold
Closes : #83
2018-10-12 00:54:39 +00:00
Bastien Nocera
21a779235d
mindtct: Fix powmax_dirs leak on error
...
A copy/paste error meant we were trying to free a variable we only just
failed to allocate.
Spotted by Seth Arnold
Closes : #81
2018-10-12 00:54:39 +00:00
Bastien Nocera
1006467f33
lib: Add accessor for minutia coordinates
...
Add fp_minutia_get_coords() so that debugging applications can show the
detected minutiae along with the captured image.
This will also help fix the positively ancient fprint_demo.
See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=907470
2018-10-08 16:50:59 +02:00
Bastien Nocera
cffe49de5c
build: Fix "already configured" meson warning
2018-10-08 16:50:59 +02:00
Bastien Nocera
7962d8cdab
build: Fix build with no drivers using imaging_dep
...
Closes : #109
2018-09-28 15:59:45 +02:00
Bastien Nocera
36dba33808
build: Test builds with a single driver with no deps
...
So that we can be sure that the build doesn't break when those drivers
are not built.
2018-09-28 15:55:56 +02:00
Bastien Nocera
c273908471
lib: Tidy fp_internal.h
...
Group global variables, structure definitions, and functions, and
group functions by which C source they're defined in.
This also removes the FP_IMGDRV_SUPPORTS_UNCONDITIONAL_CAPTURE
definition that's already available in the drivers API-accessible
fpi-core.h
2018-09-28 15:12:39 +02:00
Bastien Nocera
bf8661bf45
lib: Include "fpi-dev.h" in drivers header
2018-09-28 15:12:39 +02:00
Bastien Nocera
499b522183
lib: Remove titles from headers
...
The purpose of the headers is (or will be) explained in more details
inside each one of the matching C files.
2018-09-28 15:12:39 +02:00
Bastien Nocera
13bfe7a65c
lib: Move driver definitions to fpi-core.h
2018-09-28 15:12:39 +02:00
Bastien Nocera
e90ec04441
lib: Move fp_minutiae to fpi-img.h
2018-09-28 15:12:39 +02:00
Bastien Nocera
a59a5caaf3
lib: Add missing includes
...
Headers for size_t and ssize_t, as used in the header itself.
2018-09-28 15:12:39 +02:00