diff options
author | Richard Hughes <richard@hughsie.com> | 2011-01-15 11:23:45 +0000 |
---|---|---|
committer | Richard Hughes <richard@hughsie.com> | 2011-01-15 11:23:45 +0000 |
commit | 188891149c8488c97d2f3f58fee3b06becc5202f (patch) | |
tree | da06a9881b6ee255457c2c9ac9ddffa44b129ee4 /src/cd-device-db.c | |
parent | 3cf48cfa0b38d6c2c099bd8b2b7268cda5d740c6 (diff) | |
download | colord-188891149c8488c97d2f3f58fee3b06becc5202f.tar.gz |
Fix a small memory leak in CdDeviceDb
Diffstat (limited to 'src/cd-device-db.c')
-rw-r--r-- | src/cd-device-db.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/src/cd-device-db.c b/src/cd-device-db.c index 0b865f3..d8626ae 100644 --- a/src/cd-device-db.c +++ b/src/cd-device-db.c @@ -223,18 +223,19 @@ cd_device_db_remove (CdDeviceDb *ddb, { gboolean ret = TRUE; gchar *error_msg = NULL; - gchar *statement; + gchar *statement1 = NULL; + gchar *statement2 = NULL; gint rc; g_return_val_if_fail (CD_IS_DEVICE_DB (ddb), FALSE); g_return_val_if_fail (ddb->priv->db != NULL, FALSE); /* remove the entry */ - g_debug ("remove device %s", device_id); - statement = g_strdup_printf ("DELETE FROM devices WHERE " + g_debug ("CdDeviceDb: remove device %s", device_id); + statement1 = g_strdup_printf ("DELETE FROM devices WHERE " "device_id = '%s';", device_id); - rc = sqlite3_exec (ddb->priv->db, statement, NULL, NULL, &error_msg); + rc = sqlite3_exec (ddb->priv->db, statement1, NULL, NULL, &error_msg); if (rc != SQLITE_OK) { g_set_error (error, CD_MAIN_ERROR, @@ -245,10 +246,10 @@ cd_device_db_remove (CdDeviceDb *ddb, ret = FALSE; goto out; } - statement = g_strdup_printf ("DELETE FROM properties WHERE " + statement2 = g_strdup_printf ("DELETE FROM properties WHERE " "device_id = '%s';", device_id); - rc = sqlite3_exec (ddb->priv->db, statement, NULL, NULL, &error_msg); + rc = sqlite3_exec (ddb->priv->db, statement2, NULL, NULL, &error_msg); if (rc != SQLITE_OK) { g_set_error (error, CD_MAIN_ERROR, @@ -260,7 +261,8 @@ cd_device_db_remove (CdDeviceDb *ddb, goto out; } out: - g_free (statement); + g_free (statement1); + g_free (statement2); return ret; } |