Free loop is correct in my opinion but kfree should be extended by checkingof NULL pointer because allocation of pdpram_blk could fail and we free alsopdpram_blk->pbuffer.>> Also there should be spaces before and after the ">=".>> regards,> dan carpenter>>> + return STATUS_FAILURE;>> }>>>>