]> git.ipfire.org Git - thirdparty/bird.git/commitdiff
Move check for NULL before usage of variable.
authorOndrej Zajicek <santiago@crfreenet.org>
Sun, 11 Jan 2009 09:51:54 +0000 (10:51 +0100)
committerOndrej Zajicek <santiago@crfreenet.org>
Sun, 11 Jan 2009 09:51:54 +0000 (10:51 +0100)
proto/ospf/packet.c

index ee3528270438644d7d71e73b71026aba542d7525..5d47c48effa8338cfd83d88710b0005467a28df8 100644 (file)
@@ -232,6 +232,12 @@ ospf_rx_hook(sock * sk, int size)
 
   ps = (struct ospf_packet *) ipv4_skip_header(sk->rbuf, &size);
 
+  if (ps == NULL)
+  {
+    log(L_ERR "%s%I - bad IP header", mesg, sk->faddr);
+    return 1;
+  }
+
   if ((ifa->oa->areaid != 0) && (ntohl(ps->areaid) == 0))
   {
     WALK_LIST(iff, po->iface_list)
@@ -247,11 +253,6 @@ ospf_rx_hook(sock * sk, int size)
   DBG("%s: RX_Hook called on interface %s.\n", p->name, sk->iface->name);
 
   osize = ntohs(ps->length);
-  if (ps == NULL)
-  {
-    log(L_ERR "%s%I - bad IP header", mesg, sk->faddr);
-    return 1;
-  }
 
   if ((unsigned) size < sizeof(struct ospf_packet))
   {