summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCyrille Bagard <nocbos@gmail.com>2016-03-16 23:32:29 (GMT)
committerCyrille Bagard <nocbos@gmail.com>2016-03-16 23:32:29 (GMT)
commit4021e5f02ce71b35423852cd89fb6f0f2776f7b8 (patch)
tree8f0bbd1632d05d586f7d4af71a80a02906efd5c3
parentb94be85e00d7770af4de93bbba91b421cfb6faea (diff)
Created a new feature and stored all main moves in the database.
-rw-r--r--ChangeLog46
-rw-r--r--plugins/pychrysa/gtkext/viewpanel.c3
-rw-r--r--src/analysis/db/items/Makefile.am1
-rw-r--r--src/analysis/db/items/bookmark.c2
-rw-r--r--src/analysis/db/items/move.c782
-rw-r--r--src/analysis/db/items/move.h99
-rw-r--r--src/analysis/db/items/switcher.c6
-rw-r--r--src/analysis/db/protocol.h1
-rw-r--r--src/analysis/project.c2
-rw-r--r--src/arch/vmpa.h5
-rw-r--r--src/core/collections.c4
-rw-r--r--src/gtkext/gtkviewpanel-int.h2
-rw-r--r--src/gtkext/gtkviewpanel.c58
-rw-r--r--src/gtkext/gtkviewpanel.h11
-rw-r--r--src/gui/editem.c41
-rw-r--r--src/gui/menus/binary.c2
-rw-r--r--src/gui/menus/edition.c6
-rw-r--r--src/gui/panels/bookmarks.c4
-rw-r--r--src/gui/panels/strings.c6
-rw-r--r--src/gui/panels/symbols.c2
-rw-r--r--src/gui/tb/portions.c2
21 files changed, 1068 insertions, 17 deletions
diff --git a/ChangeLog b/ChangeLog
index 8c8ee30..63b2a01 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,49 @@
+16-03-17 Cyrille Bagard <nocbos@gmail.com>
+
+ * plugins/pychrysa/gtkext/viewpanel.c:
+ Update code.
+
+ * src/analysis/db/items/Makefile.am:
+ Add the new 'move.[ch]' files to libanalysisdbitems_la_SOURCES.
+
+ * src/analysis/db/items/bookmark.c:
+ Typo.
+
+ * src/analysis/db/items/move.c:
+ * src/analysis/db/items/move.h:
+ New entries: create a new feature and store all main moves in the database.
+
+ * src/analysis/db/items/switcher.c:
+ Typo.
+
+ * src/analysis/db/protocol.h:
+ Register DBF_MOVES as feature.
+
+ * src/analysis/project.c:
+ Update code.
+
+ * src/arch/vmpa.h:
+ Define the SQLITE_VMPA_CREATE macro to create named DB fields.
+
+ * src/core/collections.c:
+ Register a new collection for the DBF_MOVES feature.
+
+ * src/gtkext/gtkviewpanel-int.h:
+ * src/gtkext/gtkviewpanel.c:
+ * src/gtkext/gtkviewpanel.h:
+ Define the first steps of an unique interface for dealing with the caret.
+
+ * src/gui/editem.c:
+ Centralize all caret move creations for the database here.
+
+ * src/gui/menus/binary.c:
+ * src/gui/menus/edition.c:
+ * src/gui/panels/bookmarks.c:
+ * src/gui/panels/strings.c:
+ * src/gui/panels/symbols.c:
+ * src/gui/tb/portions.c:
+ Update code.
+
16-03-15 Cyrille Bagard <nocbos@gmail.com>
* plugins/readdex/class.c:
diff --git a/plugins/pychrysa/gtkext/viewpanel.c b/plugins/pychrysa/gtkext/viewpanel.c
index 8c85fbd..c031627 100644
--- a/plugins/pychrysa/gtkext/viewpanel.c
+++ b/plugins/pychrysa/gtkext/viewpanel.c
@@ -126,7 +126,8 @@ static PyObject *py_view_panel_scroll_to_address(PyObject *self, PyObject *args)
panel = GTK_VIEW_PANEL(pygobject_get(self));
- gtk_view_panel_scroll_to_address(panel, addr, SPT_RAW);
+ // FIXME
+ //gtk_view_panel_scroll_to_address(panel, addr, SPT_RAW);
Py_RETURN_NONE;
diff --git a/src/analysis/db/items/Makefile.am b/src/analysis/db/items/Makefile.am
index c0e6b47..d7acd3d 100644
--- a/src/analysis/db/items/Makefile.am
+++ b/src/analysis/db/items/Makefile.am
@@ -4,6 +4,7 @@ noinst_LTLIBRARIES = libanalysisdbitems.la
libanalysisdbitems_la_SOURCES = \
bookmark.h bookmark.c \
comment.h comment.c \
+ move.h move.c \
switcher.h switcher.c
libanalysisdbitems_la_LIBADD =
diff --git a/src/analysis/db/items/bookmark.c b/src/analysis/db/items/bookmark.c
index c8da383..be75d0f 100644
--- a/src/analysis/db/items/bookmark.c
+++ b/src/analysis/db/items/bookmark.c
@@ -1,6 +1,6 @@
/* Chrysalide - Outil d'analyse de fichiers binaires
- * bookmark.h - prototypes pour la gestion des signets au sein d'un binaire
+ * bookmark.c - gestion des signets au sein d'un binaire
*
* Copyright (C) 2014 Cyrille Bagard
*
diff --git a/src/analysis/db/items/move.c b/src/analysis/db/items/move.c
new file mode 100644
index 0000000..20b9609
--- /dev/null
+++ b/src/analysis/db/items/move.c
@@ -0,0 +1,782 @@
+
+/* Chrysalide - Outil d'analyse de fichiers binaires
+ * move.c - mémorisation des déplacements au sein de code
+ *
+ * Copyright (C) 2016 Cyrille Bagard
+ *
+ * This file is part of Chrysalide.
+ *
+ * OpenIDA 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.
+ *
+ * OpenIDA 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 Foobar. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+
+#include "move.h"
+
+
+#include <assert.h>
+#include <stdio.h>
+#include <sys/socket.h>
+
+
+#include <i18n.h>
+
+
+#include "../collection-int.h"
+#include "../item-int.h"
+#include "../../../common/io.h"
+#include "../../../gui/editem.h"
+#include "../../../gtkext/gtkviewpanel.h"
+
+
+
+
+
+/* --------------------- ELABORATION D'UN ELEMENT DE COLLECTION --------------------- */
+
+
+/* Déplacement d'un emplacement à un autre dans du code (instance) */
+struct _GDbMove
+{
+ GDbItem parent; /* A laisser en premier */
+
+ vmpa2t src; /* Emplacement de départ */
+ vmpa2t dest; /* Emplacement d'arrivée */
+
+};
+
+/* Déplacement d'un emplacement à un autre dans du code (classe) */
+struct _GDbMoveClass
+{
+ GDbItemClass parent; /* A laisser en premier */
+
+};
+
+
+
+/* Initialise la classe des déplacements au sein de codes. */
+static void g_db_move_class_init(GDbMoveClass *);
+
+/* Initialise une bascule d'affichage pour opérande numérique. */
+static void g_db_move_init(GDbMove *);
+
+/* Supprime toutes les références externes. */
+static void g_db_move_dispose(GDbMove *);
+
+/* Procède à la libération totale de la mémoire. */
+static void g_db_move_finalize(GDbMove *);
+
+/* Effectue la comparaison entre deux changements de position. */
+static gint g_db_move_cmp(GDbMove *, GDbMove *, bool);
+
+/* Importe la définition d'un déplacement depuis un flux réseau. */
+static bool g_db_move_recv_from_fd(GDbMove *, int, int);
+
+/* Exporte la définition d'un déplacement dans un flux réseau. */
+static bool g_db_move_send_to_fd(const GDbMove *, int, int);
+
+/* Construit la description humaine d'un déplacement. */
+static void g_db_move_build_label(GDbMove *);
+
+/* Exécute un déplacement dans une zone de code. */
+static bool g_db_move_run(const GDbMove *, const vmpa2t *);
+
+/* Applique un déplacement au sein d'une zone de code. */
+static bool g_db_move_apply(GDbMove *, GLoadedBinary *);
+
+/* Annule un déplacement au sein d'une zone de code. */
+static bool g_db_move_cancel(GDbMove *, GLoadedBinary *);
+
+/* Constitue les champs destinés à une insertion / modification. */
+static bool g_db_move_prepare_db_statement(const GDbMove *, bound_value **, size_t *);
+
+/* Charge les valeurs utiles pour un déplacement dans du code. */
+static bool g_db_move_load(GDbMove *, const bound_value *, size_t);
+
+
+
+
+/* ---------------------- DEFINITION DE LA COLLECTION ASSOCIEE ---------------------- */
+
+
+/* Collection dédiée aux basculements d'affichage (instance) */
+struct _GMoveCollection
+{
+ GDbCollection parent; /* A laisser en premier */
+
+};
+
+/* Collection dédiée aux basculements d'affichage (classe) */
+struct _GMoveCollectionClass
+{
+ GDbCollectionClass parent; /* A laisser en premier */
+
+};
+
+
+/* Initialise la classe des collections de déplacements. */
+static void g_move_collection_class_init(GMoveCollectionClass *);
+
+/* Initialise une collection de déplacements. */
+static void g_move_collection_init(GMoveCollection *);
+
+/* Supprime toutes les références externes. */
+static void g_move_collection_dispose(GMoveCollection *);
+
+/* Procède à la libération totale de la mémoire. */
+static void g_move_collection_finalize(GMoveCollection *);
+
+/* Crée la table des basculements dans une base de données. */
+static bool g_move_collection_create_db_table(const GMoveCollection *, sqlite3 *);
+
+/* Décrit les colonnes utiles à un chargement de données. */
+static bool g_move_collection_setup_load(GMoveCollection *, bound_value **, size_t *);
+
+/* Détermine si un élément est déjà présent ou non. */
+static GDbItem *g_move_collection_has_key(GMoveCollection *, va_list *);
+
+
+
+/* ---------------------------------------------------------------------------------- */
+/* ELABORATION D'UN ELEMENT DE COLLECTION */
+/* ---------------------------------------------------------------------------------- */
+
+
+/* Indique le type défini pour un déplacement au sein de code. */
+G_DEFINE_TYPE(GDbMove, g_db_move, G_TYPE_DB_ITEM);
+
+
+/******************************************************************************
+* *
+* Paramètres : klass = classe à initialiser. *
+* *
+* Description : Initialise la classe des déplacements au sein de codes. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_db_move_class_init(GDbMoveClass *klass)
+{
+ GObjectClass *object; /* Autre version de la classe */
+ GDbItemClass *item; /* Encore une autre vision... */
+
+ object = G_OBJECT_CLASS(klass);
+
+ object->dispose = (GObjectFinalizeFunc/* ! */)g_db_move_dispose;
+ object->finalize = (GObjectFinalizeFunc)g_db_move_finalize;
+
+ item = G_DB_ITEM_CLASS(klass);
+
+ item->cmp = (cmp_db_item_fc)g_db_move_cmp;
+
+ item->recv = (recv_db_item_fc)g_db_move_recv_from_fd;
+ item->send = (send_db_item_fc)g_db_move_send_to_fd;
+
+ item->build_label = (build_item_label_fc)g_db_move_build_label;
+ item->apply = (run_item_fc)g_db_move_apply;
+ item->cancel = (run_item_fc)g_db_move_cancel;
+
+ item->prepare_stmt = (prepare_db_statement)g_db_move_prepare_db_statement;
+ item->load = (load_db_item_fc)g_db_move_load;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = instance à initialiser. *
+* *
+* Description : Initialise un déplacement au sein de code. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_db_move_init(GDbMove *move)
+{
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = instance d'objet GLib à traiter. *
+* *
+* Description : Supprime toutes les références externes. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_db_move_dispose(GDbMove *move)
+{
+ G_OBJECT_CLASS(g_db_move_parent_class)->dispose(G_OBJECT(move));
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = instance d'objet GLib à traiter. *
+* *
+* Description : Procède à la libération totale de la mémoire. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_db_move_finalize(GDbMove *move)
+{
+ G_OBJECT_CLASS(g_db_move_parent_class)->finalize(G_OBJECT(move));
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : src = emplacement de départ d'un mouvement. *
+* dest = emplacement d'arrivée du même mouvement. *
+* *
+* Description : Organise un déplacement d'une position à une autre. *
+* *
+* Retour : Déplacement mis en place ou NULL en cas d'erreur. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+GDbMove *g_db_move_new(const vmpa2t *src, const vmpa2t *dest)
+{
+ GDbMove *result; /* Instance à retourner */
+
+ result = g_object_new(G_TYPE_DB_MOVE, NULL);
+
+ copy_vmpa(&result->src, src);
+ copy_vmpa(&result->dest, dest);
+
+ return result;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : a = premier élément à analyser. *
+* b = second élément à analyser. *
+* with = précise les horodatages à prendre en compte. *
+* *
+* Description : Effectue la comparaison entre deux changements de position. *
+* *
+* Retour : Bilan de la comparaison : -1, 0 ou 1. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static gint g_db_move_cmp(GDbMove *a, GDbMove *b, bool with)
+{
+ gint result; /* Bilan de la comparaison */
+
+ result = cmp_vmpa(&a->src, &b->src);
+
+ if (result == 0)
+ result = cmp_vmpa(&a->dest, &b->dest);
+
+ if (result == 0)
+ result = G_DB_ITEM_CLASS(g_db_move_parent_class)->cmp(G_DB_ITEM(a), G_DB_ITEM(b), with);
+
+ return result;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = bascule d'affichage aux infos à charger. [OUT] *
+* fd = flux ouvert en lecture pour l'importation. *
+* flags = éventuelles options d'envoi supplémentaires. *
+* *
+* Description : Importe la définition d'un déplacement depuis un flux réseau.*
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_db_move_recv_from_fd(GDbMove *move, int fd, int flags)
+{
+ bool status; /* Bilan d'opération initiale */
+
+ status = G_DB_ITEM_CLASS(g_db_move_parent_class)->recv(G_DB_ITEM(move), fd, flags);
+ if (!status) return false;
+
+ if (!recv_vmpa(&move->src, fd, flags))
+ return false;
+
+ if (!recv_vmpa(&move->dest, fd, flags))
+ return false;
+
+ return true;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = bascule d'affichage aux infos à sauvegarder. *
+* fd = flux ouvert en écriture pour l'exportation. *
+* flags = éventuelles options d'envoi supplémentaires. *
+* *
+* Description : Exporte la définition d'un déplacement dans un flux réseau. *
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_db_move_send_to_fd(const GDbMove *move, int fd, int flags)
+{
+ bool status; /* Bilan d'opération initiale */
+
+ status = G_DB_ITEM_CLASS(g_db_move_parent_class)->send(G_DB_ITEM(move), fd, MSG_MORE | flags);
+ if (!status) return false;
+
+ if (!send_vmpa(&move->src, fd, MSG_MORE | flags))
+ return false;
+
+ if (!send_vmpa(&move->dest, fd, flags))
+ return false;
+
+ return true;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = informations de déplacement à manipuler. *
+* *
+* Description : Construit la description humaine d'un déplacement. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_db_move_build_label(GDbMove *move)
+{
+ VMPA_BUFFER(src_loc); /* Indication de position #1 */
+ VMPA_BUFFER(dest_loc); /* Indication de position #2 */
+
+ if (has_virt_addr(&move->src))
+ vmpa2_virt_to_string(&move->src, MDS_UNDEFINED, src_loc, NULL);
+ else
+ vmpa2_phys_to_string(&move->src, MDS_UNDEFINED, src_loc, NULL);
+
+ if (has_virt_addr(&move->dest))
+ vmpa2_virt_to_string(&move->dest, MDS_UNDEFINED, dest_loc, NULL);
+ else
+ vmpa2_phys_to_string(&move->dest, MDS_UNDEFINED, dest_loc, NULL);
+
+ asprintf(&G_DB_ITEM(move)->label, _("Move from %s to %s"), src_loc, dest_loc);
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = informations de déplacement à consulter. *
+* addr = localisation à placer bien en évidence. *
+* *
+* Description : Exécute un déplacement dans une zone de code. *
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_db_move_run(const GDbMove *move, const vmpa2t *addr)
+{
+ GtkViewPanel *vpanel; /* Afficheur effectif de code */
+
+ vpanel = g_editor_item_get_current_view(NULL);
+
+ if (vpanel != NULL)
+ gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+
+ return true;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = bascule d'affichage à manipuler. *
+* binary = binaire de rattachement chargé en mémoire. *
+* *
+* Description : Applique un déplacement au sein d'une zone de code. *
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_db_move_apply(GDbMove *move, GLoadedBinary *binary)
+{
+ bool result; /* Bilan à faire remonter */
+
+ result = g_db_move_run(move, &move->dest);
+
+ return result;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = informations de déplacement à consulter. *
+* binary = binaire chargé en mémoire à modifier. *
+* *
+* Description : Annule un déplacement au sein d'une zone de code. *
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_db_move_cancel(GDbMove *move, GLoadedBinary *binary)
+{
+ bool result; /* Bilan à faire remonter */
+
+ result = g_db_move_run(move, &move->src);
+
+ return result;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = base d'éléments sur laquelle s'appuyer. *
+* values = couples de champs et de valeurs à lier. [OUT] *
+* count = nombre de ces couples. [OUT] *
+* *
+* Description : Constitue les champs destinés à une insertion / modification.*
+* *
+* Retour : Etat du besoin en sauvegarde. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_db_move_prepare_db_statement(const GDbMove *move, bound_value **values, size_t *count)
+{
+ bool status; /* Bilan d'opération initiale */
+
+ status = G_DB_ITEM_CLASS(g_db_move_parent_class)->prepare_stmt(G_DB_ITEM(move), values, count);
+ if (!status) return false;
+
+ status = prepare_vmpa_db_statement(&move->src, "src", values, count);
+ if (!status) return false;
+
+ status = prepare_vmpa_db_statement(&move->dest, "dest", values, count);
+ if (!status) return false;
+
+ return true;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : move = bascule d'affichage à charger depuis les réponses. *
+* values = tableau d'éléments à consulter. *
+* count = nombre de descriptions renseignées. *
+* *
+* Description : Charge les valeurs utiles pour un déplacement dans du code. *
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_db_move_load(GDbMove *move, const bound_value *values, size_t count)
+{
+ bool result; /* Bilan à faire remonter */
+
+ result = G_DB_ITEM_CLASS(g_db_move_parent_class)->load(G_DB_ITEM(move), values, count);
+
+ result &= load_vmpa(&move->src, "src", values, count);
+
+ result &= load_vmpa(&move->dest, "dest", values, count);
+
+ return result;
+
+}
+
+
+
+/* ---------------------------------------------------------------------------------- */
+/* DEFINITION DE LA COLLECTION ASSOCIEE */
+/* ---------------------------------------------------------------------------------- */
+
+
+/* Indique le type défini pour une collection de déplacements dans du code. */
+G_DEFINE_TYPE(GMoveCollection, g_move_collection, G_TYPE_DB_COLLECTION);
+
+
+/******************************************************************************
+* *
+* Paramètres : klass = classe à initialiser. *
+* *
+* Description : Initialise la classe des collections de déplacements. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_move_collection_class_init(GMoveCollectionClass *klass)
+{
+ GObjectClass *object; /* Autre version de la classe */
+ GDbCollectionClass *collec; /* Encore une autre vision... */
+
+ object = G_OBJECT_CLASS(klass);
+
+ object->dispose = (GObjectFinalizeFunc/* ! */)g_move_collection_dispose;
+ object->finalize = (GObjectFinalizeFunc)g_move_collection_finalize;
+
+ collec = G_DB_COLLECTION_CLASS(klass);
+
+ collec->create_table = (collec_create_db_table_fc)g_move_collection_create_db_table;
+ collec->setup_load = (collec_setup_load_fc)g_move_collection_setup_load;
+ collec->has_key = (collec_has_key_fc)g_move_collection_has_key;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : collec = instance à initialiser. *
+* *
+* Description : Initialise une collection de déplacements. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_move_collection_init(GMoveCollection *collec)
+{
+ G_DB_COLLECTION(collec)->featuring = DBF_MOVES;
+ G_DB_COLLECTION(collec)->type = G_TYPE_DB_MOVE;
+ G_DB_COLLECTION(collec)->name = "Moves";
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : collec = instance d'objet GLib à traiter. *
+* *
+* Description : Supprime toutes les références externes. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_move_collection_dispose(GMoveCollection *collec)
+{
+ G_OBJECT_CLASS(g_move_collection_parent_class)->dispose(G_OBJECT(collec));
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : collec = instance d'objet GLib à traiter. *
+* *
+* Description : Procède à la libération totale de la mémoire. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void g_move_collection_finalize(GMoveCollection *collec)
+{
+ G_OBJECT_CLASS(g_move_collection_parent_class)->finalize(G_OBJECT(collec));
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : - *
+* *
+* Description : Crée une collection dédiée aux déplacements dans du code. *
+* *
+* Retour : Collection mise en place. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+GMoveCollection *g_move_collection_new(void)
+{
+ GMoveCollection *result; /* Instance à retourner */
+
+ result = g_object_new(G_TYPE_MOVE_COLLECTION, NULL);
+
+ return result;
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : collec = ensemble d'éléments spectateur des opérations. *
+* db = accès à la base de données. *
+* *
+* Description : Crée la table des basculements dans une base de données. *
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_move_collection_create_db_table(const GMoveCollection *collec, sqlite3 *db)
+{
+ char *sql; /* Requête à exécuter */
+ int ret; /* Bilan de la création */
+ char *msg; /* Message d'erreur */
+
+ sql = "CREATE TABLE Moves (" \
+ SQLITE_DB_ITEM_CREATE ", " \
+ SQLITE_VMPA_CREATE("src") ", " \
+ SQLITE_VMPA_CREATE("dest") \
+ ");";
+
+ ret = sqlite3_exec(db, sql, NULL, NULL, &msg);
+ if (ret != SQLITE_OK)
+ {
+ fprintf(stderr, "sqlite3_exec(): %s\n", msg);
+ sqlite3_free(msg);
+ }
+
+ return (ret == SQLITE_OK);
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : collec = ensemble d'éléments à consulter. *
+* values = tableau d'éléments à compléter. [OUT] *
+* count = nombre de descriptions renseignées. [OUT] *
+* *
+* Description : Décrit les colonnes utiles à un chargement de données. *
+* *
+* Retour : Bilan de l'opération. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static bool g_move_collection_setup_load(GMoveCollection *collec, bound_value **values, size_t *count)
+{
+ bool status; /* Bilan d'une préparation */
+
+ status = G_DB_COLLECTION_CLASS(g_move_collection_parent_class)->setup_load(G_DB_COLLECTION(collec), \
+ values, count);
+ if (!status) return false;
+
+ if (!setup_load_for_vmpa(NULL, "src", values, count))
+ return false;
+
+ if (!setup_load_for_vmpa(NULL, "dest", values, count))
+ return false;
+
+ return true;
+
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : collec = ensemble d'éléments à consulter. *
+* ap = clef identifiant de manière unique un élément. *
+* *
+* Description : Détermine si un élément est déjà présent ou non. *
+* *
+* Retour : Elément trouvé ou NULL si aucun. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static GDbItem *g_move_collection_has_key(GMoveCollection *collec, va_list *ap)
+{
+ GDbItem *result; /* Bilan à retourner */
+ vmpa2t *ref; /* Adresse de référence */
+ GList *items; /* Eléments déjà en place */
+ GList *iter; /* Boucle de parcours */
+ GDbMove *bm; /* Signet à consulter */
+
+ result = NULL;
+
+ ref = va_arg(ap, vmpa2t *);
+
+ items = g_db_collection_list_items(G_DB_COLLECTION(collec));
+
+ for (iter = g_list_first(items); iter != NULL && result == NULL; iter = g_list_next(iter))
+ {
+ bm = G_DB_MOVE(iter->data);
+
+ if (cmp_vmpa(&bm->src, ref) != 0)
+
+ /**
+ * Un verrou est sensé être posé, donc il n'y a pas besoin
+ * de toucher au décompte des références car l'élément trouvé
+ * ne peut pas être supprimé.
+ */
+ result = G_DB_ITEM(bm);
+
+ }
+
+ return result;
+
+}
diff --git a/src/analysis/db/items/move.h b/src/analysis/db/items/move.h
new file mode 100644
index 0000000..16a8ba2
--- /dev/null
+++ b/src/analysis/db/items/move.h
@@ -0,0 +1,99 @@
+
+/* Chrysalide - Outil d'analyse de fichiers binaires
+ * move.h - prototypes pour la mémorisation des déplacements au sein de code
+ *
+ * Copyright (C) 2016 Cyrille Bagard
+ *
+ * This file is part of Chrysalide.
+ *
+ * OpenIDA 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.
+ *
+ * OpenIDA 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 Foobar. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+
+#ifndef _ANALYSIS_DB_ITEMS_MOVE_H
+#define _ANALYSIS_DB_ITEMS_MOVE_H
+
+
+#include <glib-object.h>
+#include <sqlite3.h>
+#include <stdbool.h>
+
+
+#include "../../../arch/vmpa.h"
+
+
+
+/* --------------------- ELABORATION D'UN ELEMENT DE COLLECTION --------------------- */
+
+
+#define G_TYPE_DB_MOVE g_db_move_get_type()
+#define G_DB_MOVE(obj) (G_TYPE_CHECK_INSTANCE_CAST((obj), g_db_move_get_type(), GDbMove))
+#define G_IS_DB_MOVE(obj) (G_TYPE_CHECK_INSTANCE_TYPE((obj), g_db_move_get_type()))
+#define G_DB_MOVE_CLASS(klass) (G_TYPE_CHECK_CLASS_CAST((klass), G_TYPE_DB_MOVE, GDbMoveClass))
+#define G_IS_DB_MOVE_CLASS(klass) (G_TYPE_CHECK_CLASS_TYPE((klass), G_TYPE_DB_MOVE))
+#define G_DB_MOVE_GET_CLASS(obj) (G_TYPE_INSTANCE_GET_CLASS((obj), G_TYPE_DB_MOVE, GDbMoveClass))
+
+
+/* Déplacement d'un emplacement à un autre dans du code (instance) */
+typedef struct _GDbMove GDbMove;
+
+/* Déplacement d'un emplacement à un autre dans du code (classe) */
+typedef struct _GDbMoveClass GDbMoveClass;
+
+
+/* Indique le type défini pour un déplacement au sein de code. */
+GType g_db_move_get_type(void);
+
+/* Organise un déplacement d'une position à une autre. */
+GDbMove *g_db_move_new(const vmpa2t *, const vmpa2t *);
+
+#if 0
+/* Fournit l'adresse associée à un signet. */
+const vmpa2t *g_db_move_get_address(GDbMove *);
+
+/* Fournit le commentaire associé à un signet. */
+const char *g_db_move_get_comment(const GDbMove *);
+
+/* Définit le commentaire associé à un signet. */
+void g_db_move_set_comment(GDbMove *, const char *);
+#endif
+
+
+/* ---------------------- DEFINITION DE LA COLLECTION ASSOCIEE ---------------------- */
+
+
+#define G_TYPE_MOVE_COLLECTION g_move_collection_get_type()
+#define G_MOVE_COLLECTION(obj) (G_TYPE_CHECK_INSTANCE_CAST((obj), g_move_collection_get_type(), GMoveCollection))
+#define G_IS_MOVE_COLLECTION(obj) (G_TYPE_CHECK_INSTANCE_TYPE((obj), g_move_collection_get_type()))
+#define G_MOVE_COLLECTION_CLASS(klass) (G_TYPE_CHECK_CLASS_CAST((klass), G_TYPE_MOVE_COLLECTION, GMoveCollectionClass))
+#define G_IS_MOVE_COLLECTION_CLASS(klass) (G_TYPE_CHECK_CLASS_TYPE((klass), G_TYPE_MOVE_COLLECTION))
+#define G_MOVE_COLLECTION_GET_CLASS(obj) (G_TYPE_INSTANCE_GET_CLASS((obj), G_TYPE_MOVE_COLLECTION, GMoveCollectionClass))
+
+
+/* Collection dédiée aux basculements d'affichage (instance) */
+typedef struct _GMoveCollection GMoveCollection;
+
+/* Collection dédiée aux basculements d'affichage (classe) */
+typedef struct _GMoveCollectionClass GMoveCollectionClass;
+
+
+/* Indique le type défini pour une collection de déplacements dans du code. */
+GType g_move_collection_get_type(void);
+
+/* Crée une collection dédiée aux déplacements dans du code. */
+GMoveCollection *g_move_collection_new(void);
+
+
+
+#endif /* _ANALYSIS_DB_ITEMS_SWITCH_H */
diff --git a/src/analysis/db/items/switcher.c b/src/analysis/db/items/switcher.c
index de5fd36..40a218c 100644
--- a/src/analysis/db/items/switcher.c
+++ b/src/analysis/db/items/switcher.c
@@ -1,6 +1,6 @@
/* Chrysalide - Outil d'analyse de fichiers binaires
- * switcher.h - gestion des basculements d'affichage d'opérandes numériques
+ * switcher.c - gestion des basculements d'affichage d'opérandes numériques
*
* Copyright (C) 2015 Cyrille Bagard
*
@@ -80,7 +80,7 @@ static void g_db_switcher_finalize(GDbSwitcher *);
/* Effectue la comparaison entre deux signets de collection. */
static gint g_db_switcher_cmp(GDbSwitcher *, GDbSwitcher *, bool);
-/* Importe la définition d'un signet dans un flux réseau. */
+/* Importe la définition d'un signet depuis un flux réseau. */
static bool g_db_switcher_recv_from_fd(GDbSwitcher *, int, int);
/* Exporte la définition d'un signet dans un flux réseau. */
@@ -359,7 +359,7 @@ static gint g_db_switcher_cmp(GDbSwitcher *a, GDbSwitcher *b, bool with)
* fd = flux ouvert en lecture pour l'importation. *
* flags = éventuelles options d'envoi supplémentaires. *
* *
-* Description : Importe la définition d'une bascule d'affichage d'opérande. *
+* Description : Importe la définition d'un signet depuis un flux réseau. *
* *
* Retour : Bilan de l'opération. *
* *
diff --git a/src/analysis/db/protocol.h b/src/analysis/db/protocol.h
index ab12654..1d9e2ba 100644
--- a/src/analysis/db/protocol.h
+++ b/src/analysis/db/protocol.h
@@ -84,6 +84,7 @@ typedef enum _DBFeatures
{
DBF_BOOKMARKS, /* Signets dans le code */
DBF_COMMENTS, /* Commentaires ajoutés */
+ DBF_MOVES, /* Déplacements dans du code */
DBF_DISPLAY_SWITCHERS, /* Choix d'affichage */
DBF_COUNT
diff --git a/src/analysis/project.c b/src/analysis/project.c
index 96d7e40..cb7f4bd 100644
--- a/src/analysis/project.c
+++ b/src/analysis/project.c
@@ -584,7 +584,7 @@ void g_study_project_add_loaded_binary(GLoadedBinary *binary, GStudyProject *pro
{
range = g_binary_symbol_get_range(symbol);
- gtk_view_panel_scroll_to_address(GTK_VIEW_PANEL(widget), get_mrange_addr(range), SPT_CENTER);
+ gtk_view_panel_request_move(GTK_VIEW_PANEL(widget), get_mrange_addr(range));
}
diff --git a/src/arch/vmpa.h b/src/arch/vmpa.h
index 9905241..f20cff0 100644
--- a/src/arch/vmpa.h
+++ b/src/arch/vmpa.h
@@ -135,10 +135,15 @@ vmpa2t *string_to_vmpa_phy(const char *);
vmpa2t *string_to_vmpa_virt(const char *);
/* Définition du tronc commun pour les créations SQLite */
+
#define SQLITE_SIMPLE_VMPA_CREATE \
"phys INTEGER, " \
"virt INTEGER"
+#define SQLITE_VMPA_CREATE(bn) \
+ bn "_phys INTEGER, " \
+ bn "_virt _INTEGER"
+
/* Décrit les colonnes utiles à un chargement de données. */
bool setup_load_for_vmpa(const vmpa2t *, const char *, bound_value **, size_t *);
diff --git a/src/core/collections.c b/src/core/collections.c
index 08c97e1..5637d05 100644
--- a/src/core/collections.c
+++ b/src/core/collections.c
@@ -33,6 +33,7 @@
#include "../analysis/db/protocol.h"
#include "../analysis/db/items/bookmark.h"
#include "../analysis/db/items/comment.h"
+#include "../analysis/db/items/move.h"
#include "../analysis/db/items/switcher.h"
@@ -106,6 +107,9 @@ bool load_hard_coded_collection_definitions(void)
id = register_collection_type(G_TYPE_COMMENT_COLLECTION);
assert(id == DBF_COMMENTS);
+ id = register_collection_type(G_TYPE_MOVE_COLLECTION);
+ assert(id == DBF_MOVES);
+
id = register_collection_type(G_TYPE_SWITCHER_COLLECTION);
assert(id == DBF_DISPLAY_SWITCHERS);
diff --git a/src/gtkext/gtkviewpanel-int.h b/src/gtkext/gtkviewpanel-int.h
index 9a2f885..fcd6ffb 100644
--- a/src/gtkext/gtkviewpanel-int.h
+++ b/src/gtkext/gtkviewpanel-int.h
@@ -103,6 +103,8 @@ struct _GtkViewPanelClass
/* Signaux */
+ void (* move_request) (GtkViewPanel *, const vmpa2t *);
+
void (* caret_moved) (GtkViewPanel *, const vmpa2t *);
};
diff --git a/src/gtkext/gtkviewpanel.c b/src/gtkext/gtkviewpanel.c
index b7829c2..7dab444 100644
--- a/src/gtkext/gtkviewpanel.c
+++ b/src/gtkext/gtkviewpanel.c
@@ -126,6 +126,14 @@ static void gtk_view_panel_class_init(GtkViewPanelClass *class)
panel_class->compute_inc = gtk_view_panel_compute_scroll_inc;
+ g_signal_new("move-request",
+ GTK_TYPE_VIEW_PANEL,
+ G_SIGNAL_RUN_LAST,
+ G_STRUCT_OFFSET(GtkViewPanelClass, move_request),
+ NULL, NULL,
+ g_cclosure_marshal_VOID__POINTER,
+ G_TYPE_NONE, 1, G_TYPE_POINTER);
+
g_signal_new("caret-moved",
GTK_TYPE_VIEW_PANEL,
G_SIGNAL_RUN_LAST,
@@ -802,6 +810,36 @@ GLoadedBinary *gtk_view_panel_get_binary(const GtkViewPanel *panel)
}
+
+
+
+
+
+
+
+
+
+/******************************************************************************
+* *
+* Paramètres : panel = composant GTK à consulter. *
+* *
+* Description : Indique la position courante du curseur. *
+* *
+* Retour : Emplacement courant du curseur. *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+#include "gtkbufferview.h" // FIXME
+const vmpa2t *gtk_view_panel_get_caret_location(const GtkViewPanel *panel)
+{
+ /* FIXME */
+
+ return gtk_buffer_view_get_caret_location(GTK_BUFFER_VIEW(panel));
+
+}
+
+
/******************************************************************************
* *
* Paramètres : panel = composant GTK à manipuler. *
@@ -894,6 +932,26 @@ void _gtk_view_panel_scroll_to_address(GtkViewPanel *panel, const vmpa2t *addr,
/******************************************************************************
* *
+* Paramètres : panel = composant GTK à manipuler. *
+* addr = adresse à présenter à l'écran. *
+* *
+* Description : Demande à qui veut répondre un déplacement du curseur. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+void gtk_view_panel_request_move(GtkViewPanel *panel, const vmpa2t *addr)
+{
+ g_signal_emit_by_name(panel, "move-request", addr);
+
+}
+
+
+/******************************************************************************
+* *
* Paramètres : panel = composant GTK à consulter. *
* line = ligne de tampon où se trouve le curseur. [OUT] *
* segment = eventuel segment de ligne actif. [OUT] *
diff --git a/src/gtkext/gtkviewpanel.h b/src/gtkext/gtkviewpanel.h
index 24f0640..0943dd1 100644
--- a/src/gtkext/gtkviewpanel.h
+++ b/src/gtkext/gtkviewpanel.h
@@ -68,6 +68,9 @@ void gtk_view_panel_set_code_display(GtkViewPanel *, bool);
/* Fournit le binaire associé à la représentation. */
GLoadedBinary *gtk_view_panel_get_binary(const GtkViewPanel *);
+
+
+
/* Adaptation d'une position sur une surface */
typedef enum _ScrollPositionTweak
{
@@ -78,11 +81,19 @@ typedef enum _ScrollPositionTweak
} ScrollPositionTweak;
+/* Indique la position courante du curseur. */
+const vmpa2t *gtk_view_panel_get_caret_location(const GtkViewPanel *);
+
/* S'assure qu'une adresse donnée est visible à l'écran. */
void _gtk_view_panel_scroll_to_address(GtkViewPanel *, const vmpa2t *, ScrollPositionTweak, bool);
#define gtk_view_panel_scroll_to_address(p, a, t) _gtk_view_panel_scroll_to_address(p, a, t, true)
+
+
+/* Demande à qui veut répondre un déplacement du curseur. */
+void gtk_view_panel_request_move(GtkViewPanel *, const vmpa2t *);
+
/* Fournit des éléments liés à la position courante dans la vue. */
bool gtk_view_panel_get_position(const GtkViewPanel *, GBufferLine **, GBufferSegment **);
diff --git a/src/gui/editem.c b/src/gui/editem.c
index f4d9d71..c4295b6 100644
--- a/src/gui/editem.c
+++ b/src/gui/editem.c
@@ -26,6 +26,7 @@
#include "editem-int.h"
+#include "../analysis/db/items/move.h"
#include "../gtkext/gtkblockview.h"
@@ -57,6 +58,9 @@ static GObject *_caret_instance = NULL;
/* Suit les changements de focus des panneaux d'affichage. */
static gboolean notify_view_panel_focus_change(GtkViewPanel *, GdkEventFocus *, void *);
+/* Lance une procédure de déplacement de la position courante. */
+static void start_moving_to_address_in_view_panel(GtkViewPanel *, const vmpa2t *, void *);
+
/* Suit les changements de position dans du code d'assembleur. */
static void track_caret_address_on_view_panel(GtkViewPanel *, const vmpa2t *, void *);
@@ -296,6 +300,36 @@ static gboolean notify_view_panel_focus_change(GtkViewPanel *vpanel, GdkEventFoc
/******************************************************************************
* *
* Paramètres : vpanel = composant d'affichage parcouru. *
+* addr = adresse de destination du curseur souhaitée. *
+* data = adresse non utilisée ici. *
+* *
+* Description : Lance une procédure de déplacement de la position courante. *
+* *
+* Retour : - *
+* *
+* Remarques : - *
+* *
+******************************************************************************/
+
+static void start_moving_to_address_in_view_panel(GtkViewPanel *vpanel, const vmpa2t *addr, void *data)
+{
+ const vmpa2t *src; /* Position courante de curseur*/
+ GDbMove *move; /* Déplacement à organiser */
+ GLoadedBinary *binary; /* Binaire en cours d'étude */
+
+ src = gtk_view_panel_get_caret_location(vpanel);
+
+ move = g_db_move_new(src, addr);
+
+ binary = gtk_view_panel_get_binary(vpanel);
+ g_loaded_binary_add_to_collection(binary, DBF_MOVES, G_DB_ITEM(move));
+
+}
+
+
+/******************************************************************************
+* *
+* Paramètres : vpanel = composant d'affichage parcouru. *
* addr = nouvelle adresse du curseur courant. *
* data = adresse non utilisée ici. *
* *
@@ -372,6 +406,9 @@ void change_editor_items_current_view(GObject *ref, GtkViewPanel *vpanel)
if (_caret_instance != NULL)
{
g_signal_handlers_disconnect_by_func(_caret_instance,
+ G_CALLBACK(start_moving_to_address_in_view_panel),
+ NULL);
+ g_signal_handlers_disconnect_by_func(_caret_instance,
G_CALLBACK(track_caret_address_on_view_panel),
NULL);
g_object_unref(_caret_instance);
@@ -380,6 +417,10 @@ void change_editor_items_current_view(GObject *ref, GtkViewPanel *vpanel)
if (vpanel != NULL)
{
+ g_signal_connect(vpanel, "move-request",
+ G_CALLBACK(start_moving_to_address_in_view_panel),
+ NULL);
+
g_signal_connect(vpanel, "caret-moved",
G_CALLBACK(track_caret_address_on_view_panel),
NULL);
diff --git a/src/gui/menus/binary.c b/src/gui/menus/binary.c
index 8058799..cf9a9b7 100644
--- a/src/gui/menus/binary.c
+++ b/src/gui/menus/binary.c
@@ -128,7 +128,7 @@ static void mcb_binary_entry_points(GtkMenuItem *menuitem, GMenuBar *bar)
addr = get_address_from_gotox_dialog(dialog);
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(bar));
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
delete_vmpa(addr);
diff --git a/src/gui/menus/edition.c b/src/gui/menus/edition.c
index 99ade8c..3be2a8e 100644
--- a/src/gui/menus/edition.c
+++ b/src/gui/menus/edition.c
@@ -335,7 +335,7 @@ static void mcb_edition_goto(GtkMenuItem *menuitem, GMenuBar *bar)
addr = get_address_from_goto_dialog(dialog);
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(bar));
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
delete_vmpa(addr);
@@ -475,7 +475,7 @@ static void mcb_edition_follow_ref(GtkMenuItem *menuitem, GMenuBar *bar)
if (virt != VMPA_NO_VIRTUAL)
{
init_vmpa(&addr, VMPA_NO_PHYSICAL, virt);
- gtk_view_panel_scroll_to_address(vpanel, &addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, &addr);
}
g_object_unref(creator);
@@ -546,7 +546,7 @@ static void mcb_edition_list_xrefs(GtkMenuItem *menuitem, GMenuBar *bar)
{
addr = get_address_from_gotox_dialog(dialog);
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
delete_vmpa(addr);
diff --git a/src/gui/panels/bookmarks.c b/src/gui/panels/bookmarks.c
index 222176a..4dc1871 100644
--- a/src/gui/panels/bookmarks.c
+++ b/src/gui/panels/bookmarks.c
@@ -680,7 +680,7 @@ static void on_bookmarks_selection_change(GtkTreeSelection *selection, GBookmark
addr = g_db_bookmark_get_address(bookmark);
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(panel));
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
g_object_unref(G_OBJECT(bookmark));
@@ -1071,7 +1071,7 @@ static gboolean on_button_press_over_bookmarks(GtkWidget *widget, GdkEventButton
addr = g_db_bookmark_get_address(bookmark);
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(panel));
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
g_object_unref(G_OBJECT(bookmark));
diff --git a/src/gui/panels/strings.c b/src/gui/panels/strings.c
index ae436cf..80706c2 100644
--- a/src/gui/panels/strings.c
+++ b/src/gui/panels/strings.c
@@ -594,7 +594,7 @@ static void on_strings_selection_change(GtkTreeSelection *selection, GStringsPan
addr = get_mrange_addr(g_binary_symbol_get_range(symbol));
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(panel));
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
g_object_unref(G_OBJECT(symbol));
@@ -833,7 +833,7 @@ static gboolean on_button_event_over_strings(GtkWidget *widget, GdkEventButton *
addr = get_mrange_addr(g_binary_symbol_get_range(symbol));
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(panel));
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
g_object_unref(G_OBJECT(symbol));
@@ -1059,7 +1059,7 @@ static void mcb_strings_panel_find_refs(GtkMenuItem *menuitem, GStringsPanel *pa
addr = get_address_from_gotox_dialog(dialog);
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(panel));
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
delete_vmpa(addr);
diff --git a/src/gui/panels/symbols.c b/src/gui/panels/symbols.c
index 422c9b4..d1bca91 100644
--- a/src/gui/panels/symbols.c
+++ b/src/gui/panels/symbols.c
@@ -553,7 +553,7 @@ static void on_symbols_selection_change(GtkTreeSelection *selection, GSymbolsPan
range = g_binary_symbol_get_range(symbol);
vpanel = g_editor_item_get_current_view(G_EDITOR_ITEM(panel));
- gtk_view_panel_scroll_to_address(vpanel, get_mrange_addr(range), SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, get_mrange_addr(range));
g_object_unref(G_OBJECT(symbol));
diff --git a/src/gui/tb/portions.c b/src/gui/tb/portions.c
index 39537eb..8ff1d31 100644
--- a/src/gui/tb/portions.c
+++ b/src/gui/tb/portions.c
@@ -246,7 +246,7 @@ static void track_address_on_binary_strip(GtkBinaryStrip *strip, GEditorItem *it
addr = gtk_binary_strip_get_location(strip);
vpanel = g_editor_item_get_current_view(item);
- gtk_view_panel_scroll_to_address(vpanel, addr, SPT_CENTER);
+ gtk_view_panel_request_move(vpanel, addr);
focus_address_in_editor_items(g_editor_item_get_current_binary(item), addr, item);