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