wallet: Remove onion-decoding information from db on payment success/…

…fail. Fixes: ElementsProject#1177
ZmnSCPxj · Mar 21, 2018 · d5d3186 · d5d3186
1 parent 489d8d0
commit d5d3186
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/wallet/db.c b/wallet/db.c
@@ -245,6 +245,12 @@ char *dbmigrations[] = {
     "ALTER TABLE payments ADD failchannel BLOB;", /* erring_channel */
     "ALTER TABLE payments ADD failupdate BLOB;", /* channel_update - can be NULL*/
     /* -- Payment routing failure information ends -- */
+    /* Delete route data for already succeeded or failed payments */
+    "UPDATE payments"
+    "   SET path_secrets = NULL"
+    "     , route_nodes = NULL"
+    "     , route_channels = NULL"
+    " WHERE status <> 0;", /* PAYMENT_PENDING */
     NULL,
 };
 

diff --git a/wallet/wallet.c b/wallet/wallet.c
@@ -1706,6 +1706,16 @@ void wallet_payment_set_status(struct wallet *wallet,
 		sqlite3_bind_sha256(stmt, 2, payment_hash);
 		db_exec_prepared(wallet->db, stmt);
 	}
+	if (newstatus != PAYMENT_PENDING) {
+		stmt = db_prepare(wallet->db,
+				  "UPDATE payments"
+				  "   SET path_secrets = NULL"
+				  "     , route_nodes = NULL"
+				  "     , route_channels = NULL"
+				  " WHERE payment_hash = ?;");
+		sqlite3_bind_sha256(stmt, 1, payment_hash);
+		db_exec_prepared(wallet->db, stmt);
+	}
 }
 
 void wallet_payment_get_failinfo(const tal_t *ctx,