Hi, I found a minor bug in libfprint-0.3.0. In imgdev.c, in function fpi_imgdev_report_finger_status(), a call to dev_change_state() is made using value IMG_ACQUIRE_STATE_AWAIT_FINGER_ON from enum fp_imgdev_enroll_status whereas a value from enum fp_imgdev_state is expected. The correct value should be IMGDEV_STATE_AWAIT_FINGER_ON.
Regards. Hugo Grostabussiat
>From 33d6f9618cd3acb88163dd96f0c88ad0d687317f Mon Sep 17 00:00:00 2001 From: Hugo Grostabussiat <[email protected]> Date: Wed, 6 Oct 2010 18:18:59 +0200 Subject: [PATCH] Corrected a call to dev_change_name() in imgdev.c, where state argument wasn't a value from fp_imgdev_state enum. --- libfprint/imgdev.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libfprint/imgdev.c b/libfprint/imgdev.c index b365d74..f564db2 100644 --- a/libfprint/imgdev.c +++ b/libfprint/imgdev.c @@ -148,7 +148,7 @@ void fpi_imgdev_report_finger_status(struct fp_img_dev *imgdev, if (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: -- 1.7.2.2
_______________________________________________ fprint mailing list [email protected] http://lists.reactivated.net/mailman/listinfo/fprint
