这哥们之前还提交过一些更无语的 patch，比如这个 https://lore.kernel.org/lkml/20210407000913.2207831-1-pakki001@umn.edu/

--- a/net/rds/send.c
+++ b/net/rds/send.c
@@ -665,7 +665,7 @@ static void rds_send_remove_from_sock(struct list_head *messages, int status)
unlock_and_drop:
                spin_unlock_irqrestore(&rm->m_rs_lock, flags);
                rds_message_put(rm);
-               if (was_on_sock)
+               if (was_on_sock && rm)
                        rds_message_put(rm);
        }

上面一行都直接用 rm 了，下一行莫名其妙又加一个判断 rm 是否为空

【 在 qlogic (戒网了) 的大作中提到: 】
: 看明大一个博士生提交了一个patch，被批
: -    xxx_put(msg);
: +    if(msg)
: ...................
--
FROM 183.60.88.*