dm thin: fix dangling bio in process_deferred_bios error path
authorMike Snitzer <snitzer@redhat.com>
Fri, 28 Mar 2014 06:15:02 +0000 (02:15 -0400)
committerBen Hutchings <ben@decadent.org.uk>
Wed, 30 Apr 2014 15:23:24 +0000 (16:23 +0100)
commit fe76cd88e654124d1431bb662a0fc6e99ca811a5 upstream.

If unable to ensure_next_mapping() we must add the current bio, which
was removed from the @bios list via bio_list_pop, back to the
deferred_bios list before all the remaining @bios.

Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Acked-by: Joe Thornber <ejt@redhat.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/md/dm-thin.c

index 2c9dd2c..80f8bd5 100644 (file)
@@ -1298,9 +1298,9 @@ static void process_deferred_bios(struct pool *pool)
                 */
                if (ensure_next_mapping(pool)) {
                        spin_lock_irqsave(&pool->lock, flags);
+                       bio_list_add(&pool->deferred_bios, bio);
                        bio_list_merge(&pool->deferred_bios, &bios);
                        spin_unlock_irqrestore(&pool->lock, flags);
-
                        break;
                }
                process_bio(tc, bio);