net: sctp: add build check for sctp_sf_eat_sack_6_2/jsctp_sf_eat_sack
authorDaniel Borkmann <dborkman@redhat.com>
Wed, 13 Feb 2013 01:03:50 +0000 (01:03 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 13 Feb 2013 19:07:14 +0000 (14:07 -0500)
In order to avoid any future surprises of kernel panics due to jprobes
function mismatches (as e.g. fixed in 4cb9d6eaf85ecd: sctp: jsctp_sf_eat_sack:
fix jprobes function signature mismatch), we should check both function
types during build and scream loudly if they do not match. __same_type
resolves to __builtin_types_compatible_p, which is 1 in case both types
are the same and 0 otherwise, qualifiers are ignored. Tested by myself.

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sctp/probe.c

index 4eaaaf68e5b7a13c7af6af7ae8ced55837478518..261b7b9858a4fc208e11365673f8f5bdd0f870a2 100644 (file)
@@ -183,6 +183,13 @@ static __init int sctpprobe_init(void)
 {
        int ret = -ENOMEM;
 
+       /* Warning: if the function signature of sctp_sf_eat_sack_6_2,
+        * has been changed, you also have to change the signature of
+        * jsctp_sf_eat_sack, otherwise you end up right here!
+        */
+       BUILD_BUG_ON(__same_type(sctp_sf_eat_sack_6_2,
+                                jsctp_sf_eat_sack) == 0);
+
        init_waitqueue_head(&sctpw.wait);
        spin_lock_init(&sctpw.lock);
        if (kfifo_alloc(&sctpw.fifo, bufsize, GFP_KERNEL))