Share GObject instances between scan contexts and matches.

10 files changed, 127 insertions, 151 deletions

diff --git a/src/analysis/scan/context.c b/src/analysis/scan/context.c
index 55311c4..fc4194b 100644
--- a/src/analysis/scan/context.c
+++ b/src/analysis/scan/context.c
@@ -430,6 +430,8 @@ void g_scan_context_register_full_matches(GScanContext *context, GSearchPattern
 
     g_hash_table_insert(context->full_trackers, pattern, matches);
 
+    g_scan_matches_attach(matches, context, pattern);
+
 }
 
 
diff --git a/src/analysis/scan/matches-int.h b/src/analysis/scan/matches-int.h
index 674fe03..4e6a244 100644
--- a/src/analysis/scan/matches-int.h
+++ b/src/analysis/scan/matches-int.h
@@ -44,7 +44,13 @@ struct _GScanMatches
 {
     GObject parent;                         /* A laisser en premier        */
 
-    GSearchPattern *source;                 /* Motif d'origine recherché   */
+    /**
+     * L'aspect constant des instances marque seulement le fait que les
+     * pointeurs sont partagés avec un contexte, qui est le réel propriétaire
+     * de ces instances.
+     */
+    const GScanContext *context;            /* Contexte de rattachement    */
+    const GSearchPattern *source;           /* Motif d'origine recherché   */
 
 };
 
diff --git a/src/analysis/scan/matches.c b/src/analysis/scan/matches.c
index 3dc15f9..d83114c 100644
--- a/src/analysis/scan/matches.c
+++ b/src/analysis/scan/matches.c
@@ -84,6 +84,7 @@ static void g_scan_matches_class_init(GScanMatchesClass *klass)
 
 static void g_scan_matches_init(GScanMatches *matches)
 {
+    matches->context = NULL;
     matches->source = NULL;
 
 }
@@ -103,7 +104,10 @@ static void g_scan_matches_init(GScanMatches *matches)
 
 static void g_scan_matches_dispose(GScanMatches *matches)
 {
-    g_clear_object(&matches->source);
+    /**
+     * Contexte et source sont des instances partagées. Leur propriété n'est
+     * donc pas à modifier avec un appel à g_clear_object() ici.
+     */
 
     G_OBJECT_CLASS(g_scan_matches_parent_class)->dispose(G_OBJECT(matches));
 
@@ -131,6 +135,53 @@ static void g_scan_matches_finalize(GScanMatches *matches)
 
 /******************************************************************************
 *                                                                             *
+*  Paramètres  : matches = instance dont l'initialisation est à compléter.    *
+*                context = contexte associé au scan courant.                  *
+*                source  = lien vers le motif recherché d'origine.            *
+*                                                                             *
+*  Description : Associe des éléments de contexte à des correspondances.      *
+*                                                                             *
+*  Retour      : -                                                            *
+*                                                                             *
+*  Remarques   : Aucun transfert de propriété n'a lieu ici !                  *
+*                                                                             *
+******************************************************************************/
+
+void g_scan_matches_attach(GScanMatches *matches, const GScanContext *context, const GSearchPattern *source)
+{
+    matches->context = context;
+    matches->source = source;
+
+}
+
+
+/******************************************************************************
+*                                                                             *
+*  Paramètres  : matches = informations de correspondances à consulter.       *
+*                                                                             *
+*  Description : Fournit le contexte du scan associé aux correspondances.     *
+*                                                                             *
+*  Retour      : Contexte de scan courant.                                    *
+*                                                                             *
+*  Remarques   : -                                                            *
+*                                                                             *
+******************************************************************************/
+
+GScanContext *g_scan_bytes_matches_get_context(const GScanMatches *matches)
+{
+    GScanContext *result;                   /* Instance à retourner        */
+
+    result = (GScanContext *)matches->context;
+
+    g_object_ref(G_OBJECT(result));
+
+    return result;
+
+}
+
+
+/******************************************************************************
+*                                                                             *
 *  Paramètres  : matches = définition de correspondance à consulter.          *
 *                                                                             *
 *  Description : Indique la source du motif d'origine recherché.              *
@@ -145,7 +196,7 @@ GSearchPattern *g_scan_matches_get_source(const GScanMatches *matches)
 {
     GSearchPattern *result;                 /* Source à retourner          */
 
-    result = matches->source;
+    result = (GSearchPattern *)matches->source;
 
     g_object_ref(G_OBJECT(result));
 
diff --git a/src/analysis/scan/matches.h b/src/analysis/scan/matches.h
index 345db96..26e54ed 100644
--- a/src/analysis/scan/matches.h
+++ b/src/analysis/scan/matches.h
@@ -51,6 +51,12 @@ typedef struct _GScanMatchesClass GScanMatchesClass;
 /* Indique le type défini pour une série de correspondances identifiées. */
 GType g_scan_matches_get_type(void);
 
+/* Associe des éléments de contexte à des correspondances. */
+void g_scan_matches_attach(GScanMatches *, const GScanContext *, const GSearchPattern *);
+
+/* Fournit le contexte du scan associé aux correspondances. */
+GScanContext *g_scan_bytes_matches_get_context(const GScanMatches *);
+
 /* Indique la source du motif d'origine recherché. */
 GSearchPattern *g_scan_matches_get_source(const GScanMatches *);
 
diff --git a/src/analysis/scan/matches/bytes-int.h b/src/analysis/scan/matches/bytes-int.h
index d356208..f6239e3 100644
--- a/src/analysis/scan/matches/bytes-int.h
+++ b/src/analysis/scan/matches/bytes-int.h
@@ -37,12 +37,6 @@ struct _GScanBytesMatches
 {
     GScanMatches parent;                    /* A laisser en premier        */
 
-    GScanContext *context;                  /* Contexte de rattachement    */
-
-    // TODO : if NDEBUG ?
-    phys_t content_start;                   /* Point de début du contenu   */
-    phys_t content_end;                     /* Point de fin du contenu     */
-
     match_area_t *areas;                    /* Zones couvertes             */
     size_t count;                           /* Nombre de zones             */
 
@@ -56,9 +50,5 @@ struct _GScanBytesMatchesClass
 };
 
 
-/* Met en place une série de correspondances avec des octets. */
-bool g_scan_bytes_matches_create(GScanBytesMatches *, GSearchPattern *, GScanContext *);
-
-
 
 #endif  /* _ANALYSIS_SCAN_MATCHES_BYTES_INT_H */
diff --git a/src/analysis/scan/matches/bytes.c b/src/analysis/scan/matches/bytes.c
index c4c43bc..218121d 100644
--- a/src/analysis/scan/matches/bytes.c
+++ b/src/analysis/scan/matches/bytes.c
@@ -117,11 +117,6 @@ static void g_scan_bytes_matches_class_init(GScanBytesMatchesClass *klass)
 
 static void g_scan_bytes_matches_init(GScanBytesMatches *matches)
 {
-    matches->context = NULL;
-
-    matches->content_start = VMPA_NO_PHYSICAL;
-    matches->content_end = VMPA_NO_PHYSICAL;
-
     matches->areas = NULL;
     matches->count = 0;
 
@@ -142,8 +137,6 @@ static void g_scan_bytes_matches_init(GScanBytesMatches *matches)
 
 static void g_scan_bytes_matches_dispose(GScanBytesMatches *matches)
 {
-    //g_clear_object(&matches->context);
-
     G_OBJECT_CLASS(g_scan_bytes_matches_parent_class)->dispose(G_OBJECT(matches));
 
 }
@@ -170,8 +163,7 @@ static void g_scan_bytes_matches_finalize(GScanBytesMatches *matches)
 
 /******************************************************************************
 *                                                                             *
-*  Paramètres  : source  = lien vers le motif recherché d'origine.            *
-*                context = contexte associé au scan courant.                  *
+*  Paramètres  : -                                                            *
 *                                                                             *
 *  Description : Crée un suivi pour série de correspondances avec des octets. *
 *                                                                             *
@@ -181,87 +173,12 @@ static void g_scan_bytes_matches_finalize(GScanBytesMatches *matches)
 *                                                                             *
 ******************************************************************************/
 
-GScanMatches *g_scan_bytes_matches_new(GSearchPattern *source, GScanContext *context)
+GScanMatches *g_scan_bytes_matches_new(void)
 {
     GScanMatches *result;                   /* Structure à retourner       */
 
     result = g_object_new(G_TYPE_SCAN_BYTES_MATCHES, NULL);
 
-    if (!g_scan_bytes_matches_create(G_SCAN_BYTES_MATCHES(result), source, context))
-        g_clear_object(&result);
-
-    return result;
-
-}
-
-
-/******************************************************************************
-*                                                                             *
-*  Paramètres  : matches = instance à initialiser pleinement.                 *
-*                source  = lien vers le motif recherché d'origine.            *
-*                context = contexte associé au scan courant.                  *
-*                                                                             *
-*  Description : Met en place une série de correspondances avec des octets.   *
-*                                                                             *
-*  Retour      : Bilan de l'opération.                                        *
-*                                                                             *
-*  Remarques   : -                                                            *
-*                                                                             *
-******************************************************************************/
-
-bool g_scan_bytes_matches_create(GScanBytesMatches *matches, GSearchPattern *source, GScanContext *context)
-{
-    bool result;                            /* Bilan à retourner           */
-    GScanMatches *base;                     /* Lien vers les infos de base */
-    GBinContent *content;                   /* Contenu à manipuler         */
-    vmpa2t start;                           /* Point de début du contenu   */
-    vmpa2t end;                             /* Point de fin du contenu     */
-
-    result = true;
-
-    base = G_SCAN_MATCHES(matches);
-
-    base->source = source;
-    g_object_ref(G_OBJECT(source));
-
-    matches->context = context;
-    //g_object_ref(G_OBJECT(context));
-
-    content = g_scan_context_get_content(context);
-
-    g_binary_content_compute_start_pos(content, &start);
-    g_binary_content_compute_end_pos(content, &end);
-
-    matches->content_start = start.physical;
-    matches->content_end = end.physical;
-
-    g_object_unref(G_OBJECT(content));
-
-    return result;
-
-}
-
-
-/******************************************************************************
-*                                                                             *
-*  Paramètres  : matches = informations de correspondances à consulter.       *
-*                                                                             *
-*  Description : Fournit le contexte du scan associé aux correspondances.     *
-*                                                                             *
-*  Retour      : Contexte de scan courant.                                    *
-*                                                                             *
-*  Remarques   : -                                                            *
-*                                                                             *
-******************************************************************************/
-
-GScanContext *g_scan_bytes_matches_get_context(const GScanBytesMatches *matches)
-{
-    GScanContext *result;                   /* Instance à retourner        */
-
-    result = matches->context;
-
-    g_object_ref(G_OBJECT(result));
-
     return result;
 
 }
@@ -433,8 +350,8 @@ char *g_scan_bytes_matches_get_modifier_path(const GScanBytesMatches *matches)
 
 static void g_scan_bytes_matches_output_to_text(const GScanBytesMatches *matches, int fd)
 {
-    GBinContent *content;                   /* Contenu binaire analysé     */
     GScanMatches *base;                     /* Lien vers les infos de base */
+    GBinContent *content;                   /* Contenu binaire analysé     */
     const char *name;                       /* Désignation du motif ciblé  */
     match_area_t *iter;                     /* Boucle de parcours #1       */
     char value[2 + ULLONG_MAXLEN];          /* Impression de la position   */
@@ -444,10 +361,10 @@ static void g_scan_bytes_matches_output_to_text(const GScanBytesMatches *matches
     const bin_t *data;                      /* Accès aux données brutes    */
     phys_t k;                               /* Boucle de parcours #2       */
 
-    content = g_scan_context_get_content(matches->context);
-
     base = G_SCAN_MATCHES(matches);
 
+    content = g_scan_context_get_content(base->context);
+
     name = g_search_pattern_get_name(base->source);
 
     for_each_match_area(iter, matches->areas)
@@ -547,6 +464,7 @@ static void g_scan_bytes_matches_output_to_json(const GScanBytesMatches *matches
     unsigned int i;                         /* Boucle de parcours #1       */
     char value[4 + ULLONG_MAXLEN];          /* Impression de la position   */
     int ret;                                /* Bilan d'une conversion      */
+    GScanMatches *base;                     /* Lien vers les infos de base */
     GBinContent *content;                   /* Contenu binaire analysé     */
     match_area_t *iter;                     /* Boucle de parcours #1       */
     vmpa2t pos;                             /* Tête de lecture             */
@@ -581,7 +499,9 @@ static void g_scan_bytes_matches_output_to_json(const GScanBytesMatches *matches
 
     write(fd, "\"matches\": [\n", 13);
 
-    content = g_scan_context_get_content(matches->context);
+    base = G_SCAN_MATCHES(matches);
+
+    content = g_scan_context_get_content(base->context);
 
     for_each_match_area(iter, matches->areas)
     {
diff --git a/src/analysis/scan/matches/bytes.h b/src/analysis/scan/matches/bytes.h
index 9e046aa..50f60a4 100644
--- a/src/analysis/scan/matches/bytes.h
+++ b/src/analysis/scan/matches/bytes.h
@@ -56,10 +56,7 @@ typedef struct _GScanBytesMatchesClass GScanBytesMatchesClass;
 GType g_scan_bytes_matches_get_type(void);
 
 /* Crée un suivi pour série de correspondances avec des octets. */
-GScanMatches *g_scan_bytes_matches_new(GSearchPattern *, GScanContext *);
-
-/* Fournit le contexte du scan associé aux correspondances. */
-GScanContext *g_scan_bytes_matches_get_context(const GScanBytesMatches *);
+GScanMatches *g_scan_bytes_matches_new(void);
 
 /* Intègre une liste de correspondances vérifiées. */
 void g_scan_bytes_matches_set_list(GScanBytesMatches *, match_area_t *, size_t);
diff --git a/src/analysis/scan/patterns/token.c b/src/analysis/scan/patterns/token.c
index 76a5e66..b3c6d53 100644
--- a/src/analysis/scan/patterns/token.c
+++ b/src/analysis/scan/patterns/token.c
@@ -300,6 +300,7 @@ bool g_bytes_token_build_id(GBytesToken *token, GEngineBackend *backend)
 *                                                                             *
 *  Paramètres  : token   = définition de la bribe à manipuler.                *
 *                matches = suivi des correspondances à consolider.            *
+*                params  = paramètres des opérations de validation.           *
 *                                                                             *
 *  Description : Transforme les correspondances locales en trouvailles.       *
 *                                                                             *
@@ -309,83 +310,70 @@ bool g_bytes_token_build_id(GBytesToken *token, GEngineBackend *backend)
 *                                                                             *
 ******************************************************************************/
 
-void g_bytes_token_check(const GBytesToken *token, GScanBytesMatches *matches)
+void g_bytes_token_check(const GBytesToken *token, GScanBytesMatches *matches, scan_node_check_params_t *params)
 {
-    scan_node_check_params_t params;        /* Rassemblement de paramètres */
-    vmpa2t start;                           /* Point de début du contenu   */
-    vmpa2t end;                             /* Point de fin du contenu     */
     match_area_t *area;                     /* Correspondance à valider    */
     match_area_t *next;                     /* Correspondance suivante     */
     vmpa2t pos;                             /* Tête de lecture             */
     const bin_t *byte;                      /* Octet à valider             */
 
-    /* Définition d'un contexte */
+    /* Réinitialisation */
 
-    params.context = g_scan_bytes_matches_get_context(matches);
-    params.content = g_scan_context_get_content(params.context);
-    params.allocator = g_umem_slice_new(sizeof(match_area_t));
+    // TODO: offset
 
-    g_binary_content_compute_start_pos(params.content, &start);
-    g_binary_content_compute_end_pos(params.content, &end);
+    params->initialized = false;
 
-    params.content_start = start.physical;
-    params.content_end = end.physical;
+    params->main_areas = NULL;
+    params->main_count = 0;
 
-    // offset
+    params->created_areas = NULL;
+    params->created_count = 0;
 
-    params.initialized = false;
-
-    params.main_areas = NULL;
-    params.main_count = 0;
-
-    params.created_areas = NULL;
-    params.created_count = 0;
-
-    params.kept_areas = NULL;
-    params.kept_count = 0;
+    params->kept_areas = NULL;
+    params->kept_count = 0;
 
     /* Lancement des analyses */
 
-    g_scan_token_node_check_forward(token->root, &params);
+    g_scan_token_node_check_forward(token->root, params);
 
     if (token->need_backward)
-        g_scan_token_node_check_backward(token->root, &params);
+        g_scan_token_node_check_backward(token->root, params);
 
     // REMME ? sort_and_filter_pending_matches(matches);
 
     if (token->fullword)
     {
-        for_each_match_area_safe(area, &params.main_areas, next)
+        for_each_match_area_safe(area, &params->main_areas, next)
         {
             /* Validation de l'octet précédent, s'il existe */
-            if (area->start > params.content_start)
+            if (area->start > params->content_start)
             {
                 init_vmpa(&pos, area->start - 1, VMPA_NO_VIRTUAL);
 
-                byte = g_binary_content_get_raw_access(params.content, &pos, 1);
+                byte = g_binary_content_get_raw_access(params->content, &pos, 1);
 
                 if (isalnum(*byte))
                 {
-                    del_match_area(area, &params.main_areas);
-                    assert(&params.main_count > 0);
-                    params.main_count--;
+                    del_match_area(area, &params->main_areas);
+                    assert(&params->main_count > 0);
+                    params->main_count--;
                     continue;
                 }
 
             }
 
             /* Validation de l'octet suivant, s'il existe */
-            if (area->end < params.content_end)
+            if (area->end < params->content_end)
             {
                 init_vmpa(&pos, area->end, VMPA_NO_VIRTUAL);
 
-                byte = g_binary_content_get_raw_access(params.content, &pos, 1);
+                byte = g_binary_content_get_raw_access(params->content, &pos, 1);
 
                 if (isalnum(*byte))
                 {
-                    del_match_area(area, &params.main_areas);
-                    assert(&params.main_count > 0);
-                    params.main_count--;
+                    del_match_area(area, &params->main_areas);
+                    assert(&params->main_count > 0);
+                    params->main_count--;
                     continue;
                 }
 
@@ -395,11 +383,7 @@ void g_bytes_token_check(const GBytesToken *token, GScanBytesMatches *matches)
 
     }
 
-    g_scan_bytes_matches_set_list(matches, params.main_areas, params.main_count);
-
-    g_object_unref(G_OBJECT(params.context));
-    g_object_unref(G_OBJECT(params.content));
-    //g_object_unref(G_OBJECT(params.allocator));
+    g_scan_bytes_matches_set_list(matches, params->main_areas, params->main_count);
 
 }
 
diff --git a/src/analysis/scan/patterns/token.h b/src/analysis/scan/patterns/token.h
index 3bc5cb7..f5b78f6 100644
--- a/src/analysis/scan/patterns/token.h
+++ b/src/analysis/scan/patterns/token.h
@@ -65,7 +65,7 @@ bool g_bytes_token_enroll(GBytesToken *, GEngineBackend *, size_t);
 bool g_bytes_token_build_id(GBytesToken *, GEngineBackend *);
 
 /* Transforme les correspondances locales en trouvailles. */
-void g_bytes_token_check(const GBytesToken *, GScanBytesMatches *);
+void g_bytes_token_check(const GBytesToken *, GScanBytesMatches *, scan_node_check_params_t *);
 
 /* Retrouve l'origine d'une correspondance à partir d'un indice. */
 char *g_bytes_token_get_modifier_path(const GBytesToken *, size_t);
diff --git a/src/analysis/scan/rule.c b/src/analysis/scan/rule.c
index 14a6d38..05fc657 100644
--- a/src/analysis/scan/rule.c
+++ b/src/analysis/scan/rule.c
@@ -613,17 +613,34 @@ bool g_scan_rule_define_pattern_ids(GScanRule *rule, GEngineBackend *backend)
 
 void g_scan_rule_check(GScanRule *rule, GEngineBackend *backend, GScanContext *context)
 {
+    scan_node_check_params_t params;        /* Rassemblement de paramètres */
+    vmpa2t start;                           /* Point de début du contenu   */
+    vmpa2t end;                             /* Point de fin du contenu     */
     size_t i;                               /* Boucle de parcours          */
     GSearchPattern *pattern;                /* Motif à intégrer            */
-    GScanBytesMatches *matches;             /* Correspondances établies    */
+    GScanMatches *matches;                  /* Correspondances établies    */
+
+    /* Définition d'un contexte */
+
+    params.context = context;
+    params.content = g_scan_context_get_content(context);
+    params.allocator = g_umem_slice_new(sizeof(match_area_t));
+
+    g_binary_content_compute_start_pos(params.content, &start);
+    g_binary_content_compute_end_pos(params.content, &end);
+
+    params.content_start = start.physical;
+    params.content_end = end.physical;
+
+    /* Vérifications */
 
     for (i = 0; i < rule->bytes_used; i++)
     {
         pattern = rule->bytes_locals[i];
 
-        matches = g_scan_bytes_matches_new(pattern, context);
+        matches = g_scan_bytes_matches_new();
 
-        g_bytes_token_check(G_BYTES_TOKEN(pattern), matches);
+        g_bytes_token_check(G_BYTES_TOKEN(pattern), G_SCAN_BYTES_MATCHES(matches), &params);
 
         g_scan_context_register_full_matches(context, pattern, matches);
 
@@ -631,6 +648,9 @@ void g_scan_rule_check(GScanRule *rule, GEngineBackend *backend, GScanContext *c
 
     }
 
+    g_object_unref(G_OBJECT(params.content));
+    //g_object_unref(G_OBJECT(params.allocator));
+
 }