put_page() does not check for NULL pointers so SDP should check this
before calling it.

Note that this probably has spaces instead of tabs but I'm using
a web email interface and I'm not sure how to preserve tabs.
I assume the current SDP owner will apply this.

Signed-off-by: Ralph Campbell <[EMAIL PROTECTED]>

diff --git a/drivers/infiniband/ulp/sdp/sdp_main.c b/drivers/infiniband/ulp/sdp/
sdp_main.c
index 809f7b8..35c4dd3 100644
--- a/drivers/infiniband/ulp/sdp/sdp_main.c
+++ b/drivers/infiniband/ulp/sdp/sdp_main.c
@@ -1212,7 +1212,8 @@ static inline struct bzcopy_state *sdp_bz_cleanup(struct b
zcopy_state *bz)
 
        if (bz->pages) {
                for (i = bz->cur_page; i < bz->page_cnt; i++)
-                       put_page(bz->pages[i]);
+                       if (bz->pages[i])
+                               put_page(bz->pages[i]);
 
                kfree(bz->pages);
        }
_______________________________________________
ewg mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg

Reply via email to