From 15fb909bdd8ca0f37dd71da7427ea6bc6bb71cbb Mon Sep 17 00:00:00 2001 From: Cyrille Bagard Date: Mon, 22 May 2023 01:09:08 +0200 Subject: Create a placeholder for Python registered items in order to keep scans working. --- plugins/pychrysalide/analysis/scan/Makefile.am | 1 + plugins/pychrysalide/analysis/scan/func.c | 6 + plugins/pychrysalide/analysis/scan/item.c | 205 +++++++++++++++++++++++++ plugins/pychrysalide/analysis/scan/item.h | 45 ++++++ plugins/pychrysalide/analysis/scan/module.c | 2 + plugins/pychrysalide/analysis/scan/options.c | 2 +- src/analysis/scan/matches/pending.c | 2 +- 7 files changed, 261 insertions(+), 2 deletions(-) create mode 100644 plugins/pychrysalide/analysis/scan/item.c create mode 100644 plugins/pychrysalide/analysis/scan/item.h diff --git a/plugins/pychrysalide/analysis/scan/Makefile.am b/plugins/pychrysalide/analysis/scan/Makefile.am index 80cfa8c..b07154a 100644 --- a/plugins/pychrysalide/analysis/scan/Makefile.am +++ b/plugins/pychrysalide/analysis/scan/Makefile.am @@ -6,6 +6,7 @@ libpychrysaanalysisscan_la_SOURCES = \ context.h context.c \ expr.h expr.c \ func.h func.c \ + item.h item.c \ module.h module.c \ options.h options.c \ scanner.h scanner.c \ diff --git a/plugins/pychrysalide/analysis/scan/func.c b/plugins/pychrysalide/analysis/scan/func.c index 61731ec..5dd846f 100644 --- a/plugins/pychrysalide/analysis/scan/func.c +++ b/plugins/pychrysalide/analysis/scan/func.c @@ -36,6 +36,9 @@ #include +#include "item.h" + + CREATE_DYN_ABSTRACT_CONSTRUCTOR(scan_function, G_TYPE_SCAN_FUNCTION, NULL); @@ -152,6 +155,9 @@ bool ensure_python_scan_function_is_registered(void) dict = PyModule_GetDict(module); + if (!ensure_python_registered_item_is_registered()) + return false; + if (!register_class_for_pygobject(dict, G_TYPE_SCAN_FUNCTION, type, &PyGObject_Type)) return false; diff --git a/plugins/pychrysalide/analysis/scan/item.c b/plugins/pychrysalide/analysis/scan/item.c new file mode 100644 index 0000000..9e9d4d3 --- /dev/null +++ b/plugins/pychrysalide/analysis/scan/item.c @@ -0,0 +1,205 @@ + +/* Chrysalide - Outil d'analyse de fichiers binaires + * item.c - équivalent Python du fichier "analysis/scan/item.c" + * + * Copyright (C) 2022 Cyrille Bagard + * + * This file is part of Chrysalide. + * + * Chrysalide is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 3 of the License, or + * (at your option) any later version. + * + * Chrysalide is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + + +#include "item.h" + + +#include + + +#include +#include +#include +#include + + + +CREATE_DYN_CONSTRUCTOR(registered_item, G_TYPE_REGISTERED_ITEM); + +/* Initialise une instance sur la base du dérivé de GObject. */ +static int py_registered_item_init(PyObject *, PyObject *, PyObject *); + + + +/****************************************************************************** +* * +* Paramètres : self = objet à initialiser (théoriquement). * +* args = arguments fournis à l'appel. * +* kwds = arguments de type key=val fournis. * +* * +* Description : Initialise une instance sur la base du dérivé de GObject. * +* * +* Retour : 0. * +* * +* Remarques : - * +* * +******************************************************************************/ + +static int py_registered_item_init(PyObject *self, PyObject *args, PyObject *kwds) +{ + int ret; /* Bilan de lecture des args. */ + +#define REGISTERED_ITEM_DOC \ + "The *RegisteredItem* class defines the basics for evaluation" \ + " items involved into content scanning.\n" \ + "\n" \ + "Instances can be created using the following constructor:\n" \ + "\n" \ + " RegisteredItem()" + + /* Initialisation d'un objet GLib */ + + ret = forward_pygobjet_init(self); + if (ret == -1) return -1; + + return 0; + +} + + +/****************************************************************************** +* * +* Paramètres : - * +* * +* Description : Fournit un accès à une définition de type à diffuser. * +* * +* Retour : Définition d'objet pour Python. * +* * +* Remarques : - * +* * +******************************************************************************/ + +PyTypeObject *get_python_registered_item_type(void) +{ + static PyMethodDef py_registered_item_methods[] = { + { NULL } + }; + + static PyGetSetDef py_registered_item_getseters[] = { + { NULL } + }; + + static PyTypeObject py_registered_item_type = { + + PyVarObject_HEAD_INIT(NULL, 0) + + .tp_name = "pychrysalide.analysis.scan.RegisteredItem", + .tp_basicsize = sizeof(PyGObject), + + .tp_flags = Py_TPFLAGS_DEFAULT, + + .tp_doc = REGISTERED_ITEM_DOC, + + .tp_methods = py_registered_item_methods, + .tp_getset = py_registered_item_getseters, + + .tp_init = py_registered_item_init, + .tp_new = py_registered_item_new, + + }; + + return &py_registered_item_type; + +} + + +/****************************************************************************** +* * +* Paramètres : - * +* * +* Description : Prend en charge l'objet 'pychrysalide...scan.RegisteredItem'.* +* * +* Retour : Bilan de l'opération. * +* * +* Remarques : - * +* * +******************************************************************************/ + +bool ensure_python_registered_item_is_registered(void) +{ + PyTypeObject *type; /* Type Python 'RegisteredItem' */ + PyObject *module; /* Module à recompléter */ + PyObject *dict; /* Dictionnaire du module */ + + type = get_python_registered_item_type(); + + if (!PyType_HasFeature(type, Py_TPFLAGS_READY)) + { + module = get_access_to_python_module("pychrysalide.analysis.scan"); + + dict = PyModule_GetDict(module); + + if (!register_class_for_pygobject(dict, G_TYPE_REGISTERED_ITEM, type, &PyGObject_Type)) + return false; + + } + + return true; + +} + + +/****************************************************************************** +* * +* Paramètres : arg = argument quelconque à tenter de convertir. * +* dst = destination des valeurs récupérées en cas de succès. * +* * +* Description : Tente de convertir en expression d'évaluation généraliste. * +* * +* Retour : Bilan de l'opération, voire indications supplémentaires. * +* * +* Remarques : - * +* * +******************************************************************************/ + +int convert_to_registered_item(PyObject *arg, void *dst) +{ + int result; /* Bilan à retourner */ + + result = PyObject_IsInstance(arg, (PyObject *)get_python_registered_item_type()); + + switch (result) + { + case -1: + /* L'exception est déjà fixée par Python */ + result = 0; + break; + + case 0: + PyErr_SetString(PyExc_TypeError, "unable to convert the provided argument to generic scan expression"); + break; + + case 1: + *((GRegisteredItem **)dst) = G_REGISTERED_ITEM(pygobject_get(arg)); + break; + + default: + assert(false); + break; + + } + + return result; + +} diff --git a/plugins/pychrysalide/analysis/scan/item.h b/plugins/pychrysalide/analysis/scan/item.h new file mode 100644 index 0000000..b3f1661 --- /dev/null +++ b/plugins/pychrysalide/analysis/scan/item.h @@ -0,0 +1,45 @@ + +/* Chrysalide - Outil d'analyse de fichiers binaires + * item.h - prototypes pour l'équivalent Python du fichier "analysis/scan/item.h" + * + * Copyright (C) 2022 Cyrille Bagard + * + * This file is part of Chrysalide. + * + * Chrysalide is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 3 of the License, or + * (at your option) any later version. + * + * Chrysalide is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + + +#ifndef _PLUGINS_PYCHRYSALIDE_ANALYSIS_SCAN_ITEM_H +#define _PLUGINS_PYCHRYSALIDE_ANALYSIS_SCAN_ITEM_H + + +#include +#include + + + +/* Fournit un accès à une définition de type à diffuser. */ +PyTypeObject *get_python_registered_item_type(void); + +/* Prend en charge l'objet 'pychrysalide.analysis.scan.RegisteredItem'. */ +bool ensure_python_registered_item_is_registered(void); + +/* Tente de convertir en expression d'évaluation généraliste. */ +int convert_to_registered_item(PyObject *, void *); + + + +#endif /* _PLUGINS_PYCHRYSALIDE_ANALYSIS_SCAN_ITEM_H */ diff --git a/plugins/pychrysalide/analysis/scan/module.c b/plugins/pychrysalide/analysis/scan/module.c index 48b7100..3af4af2 100644 --- a/plugins/pychrysalide/analysis/scan/module.c +++ b/plugins/pychrysalide/analysis/scan/module.c @@ -31,6 +31,7 @@ #include "context.h" #include "expr.h" #include "func.h" +#include "item.h" #include "options.h" #include "scanner.h" #include "space.h" @@ -107,6 +108,7 @@ bool populate_analysis_scan_module(void) if (result) result = ensure_python_scan_context_is_registered(); if (result) result = ensure_python_scan_expression_is_registered(); if (result) result = ensure_python_scan_function_is_registered(); + if (result) result = ensure_python_registered_item_is_registered(); if (result) result = ensure_python_scan_options_is_registered(); if (result) result = ensure_python_scan_namespace_is_registered(); diff --git a/plugins/pychrysalide/analysis/scan/options.c b/plugins/pychrysalide/analysis/scan/options.c index c3b29e9..ddced33 100644 --- a/plugins/pychrysalide/analysis/scan/options.c +++ b/plugins/pychrysalide/analysis/scan/options.c @@ -74,7 +74,7 @@ static int py_scan_options_init(PyObject *self, PyObject *args, PyObject *kwds) #define SCAN_OPTIONS_DOC \ "The *ScanOptions* class stores all parameters used to tune" \ - " a scanning process..\n" \ + " a scanning process.\n" \ "\n" \ "Instances can be created using the following constructor:\n" \ "\n" \ diff --git a/src/analysis/scan/matches/pending.c b/src/analysis/scan/matches/pending.c index 700b868..73b7a06 100644 --- a/src/analysis/scan/matches/pending.c +++ b/src/analysis/scan/matches/pending.c @@ -96,7 +96,7 @@ bool find_target_in_pending_matches(pending_matches_t *matches, phys_t start, si size_t i; /* Boucle de parcours */ match_area_t *area; /* Zone à initialiser */ - assert(mindex <= matches->used); + assert(*target <= matches->used); result = false; -- cgit v0.11.2-87-g4458