mirror of
https://gitlab.com/mishakmak/pam-fprint-grosshack.git
synced 2026-04-08 20:03:34 +02:00
Remove all use of g_error()
Otherwise we get bug reports about crashers that aren't crashers.
This commit is contained in:
@ -31,8 +31,10 @@ static void create_manager(void)
|
|||||||
GError *error = NULL;
|
GError *error = NULL;
|
||||||
|
|
||||||
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
||||||
if (connection == NULL)
|
if (connection == NULL) {
|
||||||
g_error("Failed to connect to session bus: %s", error->message);
|
g_print("Failed to connect to session bus: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
manager = dbus_g_proxy_new_for_name(connection,
|
manager = dbus_g_proxy_new_for_name(connection,
|
||||||
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
||||||
@ -47,14 +49,18 @@ static void delete_fingerprints(DBusGProxy *dev, const char *username)
|
|||||||
|
|
||||||
p = dbus_g_proxy_new_from_proxy(dev, "org.freedesktop.DBus.Properties", NULL);
|
p = dbus_g_proxy_new_from_proxy(dev, "org.freedesktop.DBus.Properties", NULL);
|
||||||
if (!dbus_g_proxy_call (p, "GetAll", &error, G_TYPE_STRING, "net.reactivated.Fprint.Device", G_TYPE_INVALID,
|
if (!dbus_g_proxy_call (p, "GetAll", &error, G_TYPE_STRING, "net.reactivated.Fprint.Device", G_TYPE_INVALID,
|
||||||
dbus_g_type_get_map ("GHashTable", G_TYPE_STRING, G_TYPE_VALUE), &props, G_TYPE_INVALID))
|
dbus_g_type_get_map ("GHashTable", G_TYPE_STRING, G_TYPE_VALUE), &props, G_TYPE_INVALID)) {
|
||||||
g_error("GetAll on the Properties interface failed: %s", error->message);
|
g_print("GetAll on the Properties interface failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_delete_enrolled_fingers(dev, username, &error)) {
|
if (!net_reactivated_Fprint_Device_delete_enrolled_fingers(dev, username, &error)) {
|
||||||
if (dbus_g_error_has_name (error, "net.reactivated.Fprint.Error.NoEnrolledPrints") == FALSE)
|
if (dbus_g_error_has_name (error, "net.reactivated.Fprint.Error.NoEnrolledPrints") == FALSE) {
|
||||||
g_error("ListEnrolledFingers failed: %s", error->message);
|
g_print("ListEnrolledFingers failed: %s\n", error->message);
|
||||||
else
|
exit (1);
|
||||||
|
} else {
|
||||||
g_print ("No fingerprints to delete on %s\n", g_value_get_string (g_hash_table_lookup (props, "name")));
|
g_print ("No fingerprints to delete on %s\n", g_value_get_string (g_hash_table_lookup (props, "name")));
|
||||||
|
}
|
||||||
} else {
|
} else {
|
||||||
g_print ("Fingerprints deleted on %s\n", g_value_get_string (g_hash_table_lookup (props, "name")));
|
g_print ("Fingerprints deleted on %s\n", g_value_get_string (g_hash_table_lookup (props, "name")));
|
||||||
}
|
}
|
||||||
@ -69,8 +75,10 @@ static void process_devices(char **argv)
|
|||||||
char *path;
|
char *path;
|
||||||
guint i;
|
guint i;
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Manager_get_devices(manager, &devices, &error))
|
if (!net_reactivated_Fprint_Manager_get_devices(manager, &devices, &error)) {
|
||||||
g_error("list_devices failed: %s", error->message);
|
g_print("list_devices failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
if (devices->len == 0) {
|
if (devices->len == 0) {
|
||||||
g_print("No devices found\n");
|
g_print("No devices found\n");
|
||||||
|
|||||||
@ -31,8 +31,10 @@ static void create_manager(void)
|
|||||||
GError *error = NULL;
|
GError *error = NULL;
|
||||||
|
|
||||||
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
||||||
if (connection == NULL)
|
if (connection == NULL) {
|
||||||
g_error("Failed to connect to session bus: %s", error->message);
|
g_print("Failed to connect to session bus: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
manager = dbus_g_proxy_new_for_name(connection,
|
manager = dbus_g_proxy_new_for_name(connection,
|
||||||
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
||||||
@ -45,8 +47,10 @@ static DBusGProxy *open_device(const char *username)
|
|||||||
gchar *path;
|
gchar *path;
|
||||||
DBusGProxy *dev;
|
DBusGProxy *dev;
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Manager_get_default_device(manager, &path, &error))
|
if (!net_reactivated_Fprint_Manager_get_default_device(manager, &path, &error)) {
|
||||||
g_error("list_devices failed: %s", error->message);
|
g_print("list_devices failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
if (path == NULL) {
|
if (path == NULL) {
|
||||||
g_print("No devices found\n");
|
g_print("No devices found\n");
|
||||||
@ -61,8 +65,10 @@ static DBusGProxy *open_device(const char *username)
|
|||||||
|
|
||||||
g_free (path);
|
g_free (path);
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_claim(dev, username, &error))
|
if (!net_reactivated_Fprint_Device_claim(dev, username, &error)) {
|
||||||
g_error("failed to claim device: %s", error->message);
|
g_print("failed to claim device: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
return dev;
|
return dev;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -84,8 +90,10 @@ static void do_enroll(DBusGProxy *dev)
|
|||||||
&enroll_completed, NULL);
|
&enroll_completed, NULL);
|
||||||
|
|
||||||
g_print("Enrolling right index finger.\n");
|
g_print("Enrolling right index finger.\n");
|
||||||
if (!net_reactivated_Fprint_Device_enroll_start(dev, "right-index-finger", &error))
|
if (!net_reactivated_Fprint_Device_enroll_start(dev, "right-index-finger", &error)) {
|
||||||
g_error("EnrollStart failed: %s", error->message);
|
g_print("EnrollStart failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
while (!enroll_completed)
|
while (!enroll_completed)
|
||||||
g_main_context_iteration(NULL, TRUE);
|
g_main_context_iteration(NULL, TRUE);
|
||||||
@ -93,15 +101,19 @@ static void do_enroll(DBusGProxy *dev)
|
|||||||
dbus_g_proxy_disconnect_signal(dev, "EnrollStatus",
|
dbus_g_proxy_disconnect_signal(dev, "EnrollStatus",
|
||||||
G_CALLBACK(enroll_result), &enroll_completed);
|
G_CALLBACK(enroll_result), &enroll_completed);
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_enroll_stop(dev, &error))
|
if (!net_reactivated_Fprint_Device_enroll_stop(dev, &error)) {
|
||||||
g_error("VerifyStop failed: %s", error->message);
|
g_print("VerifyStop failed: %s\n", error->message);
|
||||||
|
exit(1);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void release_device(DBusGProxy *dev)
|
static void release_device(DBusGProxy *dev)
|
||||||
{
|
{
|
||||||
GError *error = NULL;
|
GError *error = NULL;
|
||||||
if (!net_reactivated_Fprint_Device_release(dev, &error))
|
if (!net_reactivated_Fprint_Device_release(dev, &error)) {
|
||||||
g_error("ReleaseDevice failed: %s", error->message);
|
g_print("ReleaseDevice failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
int main(int argc, char **argv)
|
int main(int argc, char **argv)
|
||||||
|
|||||||
26
tests/list.c
26
tests/list.c
@ -31,8 +31,10 @@ static void create_manager(void)
|
|||||||
GError *error = NULL;
|
GError *error = NULL;
|
||||||
|
|
||||||
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
||||||
if (connection == NULL)
|
if (connection == NULL) {
|
||||||
g_error("Failed to connect to session bus: %s", error->message);
|
g_print("Failed to connect to session bus: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
manager = dbus_g_proxy_new_for_name(connection,
|
manager = dbus_g_proxy_new_for_name(connection,
|
||||||
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
||||||
@ -48,16 +50,20 @@ static void list_fingerprints(DBusGProxy *dev, const char *username)
|
|||||||
guint i;
|
guint i;
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_list_enrolled_fingers(dev, username, &fingers, &error)) {
|
if (!net_reactivated_Fprint_Device_list_enrolled_fingers(dev, username, &fingers, &error)) {
|
||||||
if (dbus_g_error_has_name (error, "net.reactivated.Fprint.Error.NoEnrolledPrints") == FALSE)
|
if (dbus_g_error_has_name (error, "net.reactivated.Fprint.Error.NoEnrolledPrints") == FALSE) {
|
||||||
g_error("ListEnrolledFingers failed: %s", error->message);
|
g_print("ListEnrolledFingers failed: %s\n", error->message);
|
||||||
else
|
exit (1);
|
||||||
|
} else {
|
||||||
fingers = NULL;
|
fingers = NULL;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
p = dbus_g_proxy_new_from_proxy(dev, "org.freedesktop.DBus.Properties", NULL);
|
p = dbus_g_proxy_new_from_proxy(dev, "org.freedesktop.DBus.Properties", NULL);
|
||||||
if (!dbus_g_proxy_call (p, "GetAll", &error, G_TYPE_STRING, "net.reactivated.Fprint.Device", G_TYPE_INVALID,
|
if (!dbus_g_proxy_call (p, "GetAll", &error, G_TYPE_STRING, "net.reactivated.Fprint.Device", G_TYPE_INVALID,
|
||||||
dbus_g_type_get_map ("GHashTable", G_TYPE_STRING, G_TYPE_VALUE), &props, G_TYPE_INVALID))
|
dbus_g_type_get_map ("GHashTable", G_TYPE_STRING, G_TYPE_VALUE), &props, G_TYPE_INVALID)) {
|
||||||
g_error("GetAll on the Properties interface failed: %s", error->message);
|
g_print("GetAll on the Properties interface failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
if (fingers == NULL || g_strv_length (fingers) == 0) {
|
if (fingers == NULL || g_strv_length (fingers) == 0) {
|
||||||
g_print("User %s has no fingers enrolled for %s.\n", username, g_value_get_string (g_hash_table_lookup (props, "name")));
|
g_print("User %s has no fingers enrolled for %s.\n", username, g_value_get_string (g_hash_table_lookup (props, "name")));
|
||||||
@ -85,8 +91,10 @@ static void process_devices(char **argv)
|
|||||||
char *path;
|
char *path;
|
||||||
guint i;
|
guint i;
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Manager_get_devices(manager, &devices, &error))
|
if (!net_reactivated_Fprint_Manager_get_devices(manager, &devices, &error)) {
|
||||||
g_error("list_devices failed: %s", error->message);
|
g_print("list_devices failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
if (devices->len == 0) {
|
if (devices->len == 0) {
|
||||||
g_print("No devices found\n");
|
g_print("No devices found\n");
|
||||||
|
|||||||
@ -36,8 +36,10 @@ static void create_manager(void)
|
|||||||
GError *error = NULL;
|
GError *error = NULL;
|
||||||
|
|
||||||
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
connection = dbus_g_bus_get(DBUS_BUS_SYSTEM, &error);
|
||||||
if (connection == NULL)
|
if (connection == NULL) {
|
||||||
g_error("Failed to connect to session bus: %s", error->message);
|
g_print("Failed to connect to session bus: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
manager = dbus_g_proxy_new_for_name(connection,
|
manager = dbus_g_proxy_new_for_name(connection,
|
||||||
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
"net.reactivated.Fprint", "/net/reactivated/Fprint/Manager",
|
||||||
@ -50,8 +52,10 @@ static DBusGProxy *open_device(const char *username)
|
|||||||
gchar *path;
|
gchar *path;
|
||||||
DBusGProxy *dev;
|
DBusGProxy *dev;
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Manager_get_default_device(manager, &path, &error))
|
if (!net_reactivated_Fprint_Manager_get_default_device(manager, &path, &error)) {
|
||||||
g_error("list_devices failed: %s", error->message);
|
g_print("list_devices failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
if (path == NULL) {
|
if (path == NULL) {
|
||||||
g_print("No devices found\n");
|
g_print("No devices found\n");
|
||||||
@ -66,8 +70,10 @@ static DBusGProxy *open_device(const char *username)
|
|||||||
|
|
||||||
g_free (path);
|
g_free (path);
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_claim(dev, username, &error))
|
if (!net_reactivated_Fprint_Device_claim(dev, username, &error)) {
|
||||||
g_error("failed to claim device: %s", error->message);
|
g_print("failed to claim device: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
return dev;
|
return dev;
|
||||||
}
|
}
|
||||||
@ -78,8 +84,10 @@ static void find_finger(DBusGProxy *dev, const char *username)
|
|||||||
char **fingers;
|
char **fingers;
|
||||||
guint i;
|
guint i;
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_list_enrolled_fingers(dev, username, &fingers, &error))
|
if (!net_reactivated_Fprint_Device_list_enrolled_fingers(dev, username, &fingers, &error)) {
|
||||||
g_error("ListEnrolledFingers failed: %s", error->message);
|
g_print("ListEnrolledFingers failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
if (fingers == NULL || g_strv_length (fingers) == 0) {
|
if (fingers == NULL || g_strv_length (fingers) == 0) {
|
||||||
g_print("No fingers enrolled for this device.\n");
|
g_print("No fingers enrolled for this device.\n");
|
||||||
@ -122,8 +130,10 @@ static void do_verify(DBusGProxy *dev)
|
|||||||
dbus_g_proxy_connect_signal(dev, "VerifyFingerSelected", G_CALLBACK(verify_finger_selected),
|
dbus_g_proxy_connect_signal(dev, "VerifyFingerSelected", G_CALLBACK(verify_finger_selected),
|
||||||
NULL, NULL);
|
NULL, NULL);
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_verify_start(dev, finger_name, &error))
|
if (!net_reactivated_Fprint_Device_verify_start(dev, finger_name, &error)) {
|
||||||
g_error("VerifyStart failed: %s", error->message);
|
g_print("VerifyStart failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
|
|
||||||
while (!verify_completed)
|
while (!verify_completed)
|
||||||
g_main_context_iteration(NULL, TRUE);
|
g_main_context_iteration(NULL, TRUE);
|
||||||
@ -131,15 +141,19 @@ static void do_verify(DBusGProxy *dev)
|
|||||||
dbus_g_proxy_disconnect_signal(dev, "VerifyStatus", G_CALLBACK(verify_result), &verify_completed);
|
dbus_g_proxy_disconnect_signal(dev, "VerifyStatus", G_CALLBACK(verify_result), &verify_completed);
|
||||||
dbus_g_proxy_disconnect_signal(dev, "VerifyFingerSelected", G_CALLBACK(verify_finger_selected), NULL);
|
dbus_g_proxy_disconnect_signal(dev, "VerifyFingerSelected", G_CALLBACK(verify_finger_selected), NULL);
|
||||||
|
|
||||||
if (!net_reactivated_Fprint_Device_verify_stop(dev, &error))
|
if (!net_reactivated_Fprint_Device_verify_stop(dev, &error)) {
|
||||||
g_error("VerifyStop failed: %s", error->message);
|
g_print("VerifyStop failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void release_device(DBusGProxy *dev)
|
static void release_device(DBusGProxy *dev)
|
||||||
{
|
{
|
||||||
GError *error = NULL;
|
GError *error = NULL;
|
||||||
if (!net_reactivated_Fprint_Device_release(dev, &error))
|
if (!net_reactivated_Fprint_Device_release(dev, &error)) {
|
||||||
g_error("ReleaseDevice failed: %s", error->message);
|
g_print("ReleaseDevice failed: %s\n", error->message);
|
||||||
|
exit (1);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static const GOptionEntry entries[] = {
|
static const GOptionEntry entries[] = {
|
||||||
|
|||||||
Reference in New Issue
Block a user