Sign in to follow this  
Followers 0
corrector

Découvertes dans le code GPL de la Freebox

3 posts in this topic

Bonjour à tous,

 

Je me demandais si des curieux avaient fait des trouvailles en parcourant le code des logiciels GPL des dernières versions du firmware des différents modèles de Freebox, codes qui sont disponibles comme chacun sait sur :

 

http://floss.freebox.fr/

 

Personne n'a vu de curiosités?

 

Ou bien personne n'a fouillé?

Share this post


Link to post
Share on other sites

Dans linux-2.6.31.7-fbx.patch :

diff -ruw linux-2.6.31.7/net/core/dev.c linux-2.6.31.7-fbx/net/core/dev.c
--- linux-2.6.31.7/net/core/dev.c    2009-12-08 20:13:50.000000000 +0100
+++ linux-2.6.31.7-fbx/net/core/dev.c    2010-10-25 13:43:58.361437288 +0200

+#if defined(CONFIG_FREEBOX_BRIDGE) || defined(CONFIG_FREEBOX_BRIDGE_MODULE)
+struct sk_buff *(*fbxbridge_handle_frame_hook)(struct fbxbridge *p, struct sk_buff *skb);
+
+struct fbxbridge;
+
+static inline struct sk_buff *handle_fbxbridge(struct sk_buff *skb,
+                           struct packet_type **pt_prev, int *ret,
+                           struct net_device *orig_dev)
+{
+    struct fbxbridge *fbxbr;
+
+    if (skb->pkt_type == PACKET_LOOPBACK ||
+        (fbxbr = skb->dev->fbx_bridge_port) == NULL)
+        return skb;
+
+    if (skb->protocol != __constant_htons(ETH_P_IP) &&
+        skb->protocol != __constant_htons(ETH_P_ARP))
+        return skb;
+
+    if (*pt_prev) {
+        *ret = deliver_skb(skb, *pt_prev, orig_dev);
+        *pt_prev = NULL;
+    }
+
+    return fbxbridge_handle_frame_hook(fbxbr, skb);
+}
+#else
+#define handle_fbxbridge(skb, pt_prev, ret, orig_dev)   (skb)
+#endif

Puisque jamais fbxbridge_handle_frame_hook ne reçoit une valeur (autre que l'initialisation implicite à null des objets globaux), il est évident que ce code ne peut pas marcher en l'état! Il n'y a même pas de test à null avant d'invoquer fbxbridge_handle_frame_hook.

 

Il s'agit donc d'un fragment.

Share this post


Link to post
Share on other sites

Suis-je le seul à trouver que la GPL n'est pas respectée par Free?

Share this post


Link to post
Share on other sites
Sign in to follow this  
Followers 0