On Wed, 2014-06-25 at 09:06 +0300, Timo Teras wrote:
> It did. Thanks. Forget the earlier patch, it's irrelevant and the
> original code is right. Seems the bug is in the generic code and
> affects all drivers that are capable of 'finger present' detection.
>
> It seems to be a bug since 2008 but mostly unnoticed due to doing only
> that one enrollment.
>
> The following should fix it:
>
> diff --git a/libfprint/imgdev.c b/libfprint/imgdev.c
> index f960ee3..3b5d3f3 100644
> --- a/libfprint/imgdev.c
> +++ b/libfprint/imgdev.c
> @@ -159,7 +159,7 @@ void fpi_imgdev_report_finger_status(struct fp_img_dev
> *imgdev,
> r > 0 && r != FP_ENROLL_COMPLETE && r != FP_ENROLL_FAIL) {
> imgdev->action_result = 0;
> imgdev->action_state =
> IMG_ACQUIRE_STATE_AWAIT_FINGER_ON;
> - dev_change_state(imgdev,
> IMG_ACQUIRE_STATE_AWAIT_FINGER_ON);
> + dev_change_state(imgdev,
> IMGDEV_STATE_AWAIT_FINGER_ON);
> }
> break;
> case IMG_ACTION_VERIFY:
This fixes it perfectly! I can vouch for this change, at least it works
both with my U.are.u 4500 (using uru4000) and my upek touchstrip (using
upekts_img) readers... Thanks a lot!
Kind regards,
Martin Hejnfelt
_______________________________________________
fprint mailing list
fprint@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/fprint
