Merge branch 'master' of http://olsr.org/git/olsrd
authorHenning Rogge <henning.rogge@fkie.fraunhofer.de>
Mon, 31 Mar 2014 09:18:51 +0000 (11:18 +0200)
committerHenning Rogge <henning.rogge@fkie.fraunhofer.de>
Mon, 31 Mar 2014 09:18:51 +0000 (11:18 +0200)
src/lq_packet.c
src/process_package.c

index e20325c..d0ca71c 100644 (file)
@@ -360,8 +360,8 @@ serialize_lq_hello(struct lq_hello_message *lq_hello, struct interface *outif)
   /*
    * Initially, we want to put the complete lq_hello into the message.
    * For this flush the output buffer (if there are some bytes in).
-   * This is a hack/fix, which prevents message fragementation resulting
-   * in instable links. The ugly lq/genmsg code should be reworked anyhow.
+   * This is a hack/fix, which prevents message fragmentation resulting
+   * in unstable links. The ugly lq/genmsg code should be reworked anyhow.
    */
   if (0 < net_output_pending(outif)) {
     for (i = 0; i <= MAX_NEIGH; i++) {
@@ -408,7 +408,7 @@ serialize_lq_hello(struct lq_hello_message *lq_hello, struct interface *outif)
         req = olsr_cnf->ipsize + olsr_sizeof_hello_lqdata();
 
         // no, we also need space for an info header, as this is the
-        // first neighbor with the current neighor type and link type
+        // first neighbor with the current neighbor type and link type
 
         if (is_first)
           req += sizeof(struct lq_hello_info_header);
@@ -552,8 +552,8 @@ serialize_lq_tc(struct lq_tc_message *lq_tc, struct interface *outif)
   /*
    * Initially, we want to put the complete lq_tc into the message.
    * For this flush the output buffer (if there are some bytes in).
-   * This is a hack/fix, which prevents message fragementation resulting
-   * in instable links. The ugly lq/genmsg code should be reworked anyhow.
+   * This is a hack/fix, which prevents message fragmentation resulting
+   * in unstable links. The ugly lq/genmsg code should be reworked anyhow.
    */
   if (0 < net_output_pending(outif)) {
     for (neigh = lq_tc->neigh; neigh != NULL; neigh = neigh->next) {
index 780efab..b39b692 100644 (file)
@@ -441,21 +441,15 @@ olsr_hello_tap(struct hello_message *message, struct interface *in_if, const uni
 
     /* find the input interface in the list of neighbor interfaces */
     for (walker = message->neighbors; walker != NULL; walker = walker->next) {
-      if (walker->link != UNSPEC_LINK
-          && ipequal(&walker->address, &in_if->ip_addr)) {
+      if (ipequal(&walker->address, &in_if->ip_addr)) {
+        /*
+         * memorize our neighbour's idea of the link quality, so that we
+         * know the link quality in both directions
+         */
+        olsr_memorize_foreign_hello_lq(lnk, walker->link != UNSPEC_LINK ? walker : NULL);
         break;
       }
     }
-
-    /*
-     * memorize our neighbour's idea of the link quality, so that we
-     * know the link quality in both directions
-     *
-     * walker is NULL if there the current interface was not included in
-     * the message (or was included as an UNSPEC_LINK)
-     */
-    olsr_memorize_foreign_hello_lq(lnk, walker);
-
     /* update packet loss for link quality calculation */
     olsr_received_hello_handler(lnk);
   }
@@ -493,7 +487,7 @@ olsr_hello_tap(struct hello_message *message, struct interface *in_if, const uni
   if (neighbor->willingness != WILL_NEVER)
     process_message_neighbors(neighbor, message);
 
-  /* Process changes immedeatly in case of MPR updates */
+  /* Process changes immediately in case of MPR updates */
   olsr_process_changes();
 
   olsr_free_hello_packet(message);