diff options
| -rw-r--r-- | ChangeLog | 56 | ||||
| -rw-r--r-- | configure.ac | 2 | ||||
| -rw-r--r-- | src/Makefile.am | 3 | ||||
| -rw-r--r-- | src/analysis/binary-int.h | 2 | ||||
| -rw-r--r-- | src/analysis/binary.c | 16 | ||||
| -rw-r--r-- | src/analysis/binary.h | 14 | ||||
| -rw-r--r-- | src/arch/dalvik/instruction.c | 45 | ||||
| -rw-r--r-- | src/glibext/gbufferline.c | 4 | ||||
| -rw-r--r-- | src/glibext/gbufferline.h | 4 | ||||
| -rw-r--r-- | src/glibext/gcodebuffer.c | 11 | ||||
| -rw-r--r-- | src/gtkext/Makefile.am | 5 | ||||
| -rwxr-xr-x | src/gtkext/graph/Makefile.am (renamed from src/graph/Makefile.am) | 6 | ||||
| -rw-r--r-- | src/gtkext/graph/dot.c (renamed from src/graph/dot.c) | 82 | ||||
| -rw-r--r-- | src/gtkext/graph/dot.h (renamed from src/graph/dot.h) | 4 | ||||
| -rw-r--r-- | src/gtkext/graph/layout.c (renamed from src/graph/layout.c) | 34 | ||||
| -rw-r--r-- | src/gtkext/graph/layout.h (renamed from src/graph/layout.h) | 4 | ||||
| -rw-r--r-- | src/gtkext/graph/node.c (renamed from src/graph/node.c) | 78 | ||||
| -rw-r--r-- | src/gtkext/graph/node.h (renamed from src/graph/node.h) | 9 | ||||
| -rw-r--r-- | src/gtkext/gtkgraphview.c | 8 | ||||
| -rw-r--r-- | src/gtkext/gtklinkrenderer.c | 100 | ||||
| -rw-r--r-- | src/gtkext/gtklinkrenderer.h | 14 | ||||
| -rw-r--r-- | src/gtkext/gtkviewpanel.c | 7 | ||||
| -rw-r--r-- | src/project.c | 4 | 
23 files changed, 427 insertions, 85 deletions
| @@ -1,3 +1,59 @@ +12-11-02  Cyrille Bagard <nocbos@gmail.com> + +	* configure.ac: +	Move the Makefile of the graph directory. + +	* src/analysis/binary.c: +	* src/analysis/binary.h: +	* src/analysis/binary-int.h: +	Define display options for each kind of view. + +	* src/arch/dalvik/instruction.c: +	Provide links from Dalvik instructions. + +	* src/glibext/gbufferline.c: +	* src/glibext/gbufferline.h: +	Define COL_MARGIN in the header file. + +	* src/glibext/gcodebuffer.c: +	Compute size of buffers more accurately. Fix a bug when looking for +	a line using an address. + +	* src/graph/dot.c: +	* src/graph/dot.h: +	* src/graph/layout.c: +	* src/graph/layout.h: +	* src/graph/Makefile.am: +	* src/graph/node.c: +	* src/graph/node.h: +	Moved entries. + +	* src/gtkext/graph/dot.c: +	* src/gtkext/graph/dot.h: +	* src/gtkext/graph/layout.c: +	* src/gtkext/graph/layout.h: +	* src/gtkext/graph/Makefile.am: +	* src/gtkext/graph/node.c: +	* src/gtkext/graph/node.h: +	Improve the computing and the rendering of the graphic view. + +	* src/gtkext/gtkgraphview.c: +	* src/gtkext/gtklinkrenderer.c: +	* src/gtkext/gtklinkrenderer.h: +	Improve the computing and the rendering of the graphic view. + +	* src/gtkext/gtkviewpanel.c: +	Fix a bug: use the real requested size of a panel. + +	* src/gtkext/Makefile.am: +	Add graph/libgtkextgraph.la to libgtkext_la_LIBADD. + +	* src/Makefile.am: +	Remove all references to the graph directory. + +	* src/project.c: +	Update code. +  12-10-28  Cyrille Bagard <nocbos@gmail.com>  	* src/arch/archbase.c: diff --git a/configure.ac b/configure.ac index 0a79182..5ccb4af 100644 --- a/configure.ac +++ b/configure.ac @@ -288,8 +288,8 @@ AC_CONFIG_FILES([Makefile                   src/format/mangling/Makefile                   src/format/pe/Makefile                   src/glibext/Makefile -                 src/graph/Makefile                   src/gtkext/Makefile +                 src/gtkext/graph/Makefile                   src/gui/Makefile                   src/gui/menus/Makefile                   src/gui/panels/Makefile diff --git a/src/Makefile.am b/src/Makefile.am index 2d6fd46..8eff972 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -43,7 +43,6 @@ libchrysagtkext_la_LDFLAGS = $(LIBGTK_LIBS) $(LIBGRAPH_LIBS)		\  	-L.libs -lchrysadisass  libchrysagtkext_la_LIBADD =						\ -	graph/libgraph.la							\  	gtkext/libgtkext.la @@ -107,6 +106,6 @@ chrysalide_LDADD = $(LIBINTL) 				\  # gtkext doit être traité en premier, à cause des marshals GLib -SUBDIRS = glibext gtkext analysis arch format common debug decomp dialogs graph gui plugins +SUBDIRS = glibext gtkext analysis arch format common debug decomp dialogs gui plugins  # TODO: rm -rf panels  diff --git a/src/analysis/binary-int.h b/src/analysis/binary-int.h index c9be0bf..f813aa8 100644 --- a/src/analysis/binary-int.h +++ b/src/analysis/binary-int.h @@ -63,7 +63,7 @@ struct _GLoadedBinary      size_t decbuf_count;                    /* Taille des tableaux         */      size_t defsrc;                          /* Fichier source principal    */ -    bool text_display[2];                   /* Position et code binaire #1 */ +    bool text_display[BDT_COUNT][2];        /* Position et code binaire #1 */      bool lines_display;                     /* Affichage des lignes        */  }; diff --git a/src/analysis/binary.c b/src/analysis/binary.c index d5bf029..b9292aa 100644 --- a/src/analysis/binary.c +++ b/src/analysis/binary.c @@ -122,8 +122,10 @@ static void g_loaded_binary_class_init(GLoadedBinaryClass *klass)  static void g_loaded_binary_init(GLoadedBinary *binary)  { -    binary->text_display[0] = true; -    binary->text_display[1] = true; +    binary->text_display[BDT_ASM][0] = true; +    binary->text_display[BDT_ASM][1] = true; +    binary->text_display[BDT_GRAPH][0] = true; +    binary->text_display[BDT_GRAPH][1] = false;      binary->lines_display = true; @@ -681,6 +683,7 @@ GCodeBuffer *g_loaded_binary_get_disassembled_buffer(const GLoadedBinary *binary  /******************************************************************************  *                                                                             *  *  Paramètres  : binary = élément binaire à consulter.                        * +*                type   = type de représentation visée.                       *  *                                                                             *  *  Description : Indique si les adresses doivent apparaître dans le rendu.    *  *                                                                             * @@ -690,9 +693,9 @@ GCodeBuffer *g_loaded_binary_get_disassembled_buffer(const GLoadedBinary *binary  *                                                                             *  ******************************************************************************/ -bool *g_loaded_binary_display_addresses_in_text(GLoadedBinary *binary) +bool *g_loaded_binary_display_addresses_in_text(GLoadedBinary *binary, BinaryDisplayType type)  { -    return &binary->text_display[0]; +    return &binary->text_display[type][0];  } @@ -700,6 +703,7 @@ bool *g_loaded_binary_display_addresses_in_text(GLoadedBinary *binary)  /******************************************************************************  *                                                                             *  *  Paramètres  : binary = élément binaire à consulter.                        * +*                type   = type de représentation visée.                       *  *                                                                             *  *  Description : Indique si le code doit apparaître dans le rendu.            *  *                                                                             * @@ -709,9 +713,9 @@ bool *g_loaded_binary_display_addresses_in_text(GLoadedBinary *binary)  *                                                                             *  ******************************************************************************/ -bool *g_loaded_binary_display_code_in_text(GLoadedBinary *binary) +bool *g_loaded_binary_display_code_in_text(GLoadedBinary *binary, BinaryDisplayType type)  { -    return &binary->text_display[1]; +    return &binary->text_display[type][1];  } diff --git a/src/analysis/binary.h b/src/analysis/binary.h index 7ff9080..eb042c6 100644 --- a/src/analysis/binary.h +++ b/src/analysis/binary.h @@ -62,6 +62,16 @@ typedef enum _BinaryPartModel  } BinaryPartModel; +/* Types de représentation */ +typedef enum _BinaryDisplayType +{ +    BDT_ASM,                                /* Rendu du code brut          */ +    BDT_GRAPH,                              /* Rendu avec graphiques       */ + +    BDT_COUNT + +} BinaryDisplayType; +  /* Indique le type défini pour une description de fichier binaire. */  GType g_loaded_binary_get_type(void); @@ -97,10 +107,10 @@ GArchInstruction *g_loaded_binary_get_instructions(const GLoadedBinary *);  GCodeBuffer *g_loaded_binary_get_disassembled_buffer(const GLoadedBinary *);  /* Indique si les adresses doivent apparaître dans le rendu. */ -bool *g_loaded_binary_display_addresses_in_text(GLoadedBinary *); +bool *g_loaded_binary_display_addresses_in_text(GLoadedBinary *, BinaryDisplayType);  /* Indique si le code doit apparaître dans le rendu. */ -bool *g_loaded_binary_display_code_in_text(GLoadedBinary *); +bool *g_loaded_binary_display_code_in_text(GLoadedBinary *, BinaryDisplayType);  /* Fournit le tampon associé au contenu d'un fichier source. */  GCodeBuffer *g_loaded_binary_get_decompiled_buffer(const GLoadedBinary *, size_t); diff --git a/src/arch/dalvik/instruction.c b/src/arch/dalvik/instruction.c index 49b5573..d494fb4 100644 --- a/src/arch/dalvik/instruction.c +++ b/src/arch/dalvik/instruction.c @@ -26,6 +26,7 @@  #include "instruction-int.h"  #include "decomp/translate.h" +#include "operands/target.h"  #include "../instruction-int.h" @@ -452,7 +453,49 @@ static const char *dalvik_get_instruction_text(const GDalvikInstruction *instr,  static InstructionLinkType dalvik_get_instruction_link(const GDalvikInstruction *instr, vmpa_t *addr)  { -    return ILT_NONE/*instr->get_link(instr, addr)*/; +    InstructionLinkType result;             /* Type de lien à retourner    */ +    GArchOperand *operand;                  /* Opérande à manipuler        */ +    const GImmOperand *imm;                 /* Valeur immédiate            */ + +    switch (instr->type) +    { +        case DOP_IF_EQ: +        case DOP_IF_NE: +        case DOP_IF_LT: +        case DOP_IF_GE: +        case DOP_IF_GT: +        case DOP_IF_LE: + +            operand = g_arch_instruction_get_operand(G_ARCH_INSTRUCTION(instr), 2); +            imm = g_dalvik_target_operand_get_value(G_DALVIK_TARGET_OPERAND(operand)); + +            if (g_imm_operand_to_vmpa_t(imm, addr)) result = ILT_JUMP_IF_TRUE; +            else result = ILT_NONE; + +            break; + +        case DOP_IF_EQZ: +        case DOP_IF_NEZ: +        case DOP_IF_LTZ: +        case DOP_IF_GEZ: +        case DOP_IF_GTZ: +        case DOP_IF_LEZ: + +            operand = g_arch_instruction_get_operand(G_ARCH_INSTRUCTION(instr), 1); +            imm = g_dalvik_target_operand_get_value(G_DALVIK_TARGET_OPERAND(operand)); + +            if (g_imm_operand_to_vmpa_t(imm, addr)) result = ILT_JUMP_IF_TRUE; +            else result = ILT_NONE; + +            break; + +        default: +            result = ILT_NONE; +            break; + +    } + +    return result;  } diff --git a/src/glibext/gbufferline.c b/src/glibext/gbufferline.c index 4360f8c..ce3e19f 100644 --- a/src/glibext/gbufferline.c +++ b/src/glibext/gbufferline.c @@ -35,10 +35,6 @@ -/* Espace entre les colonnes */ -#define COL_MARGIN 23 - -  /* ---------------------------- REGROUPEMENT PAR COLONNE ---------------------------- */ diff --git a/src/glibext/gbufferline.h b/src/glibext/gbufferline.h index ce75895..3cc992b 100644 --- a/src/glibext/gbufferline.h +++ b/src/glibext/gbufferline.h @@ -100,9 +100,13 @@ typedef enum _RenderingTagType  #define RTT_NONE RTT_RAW    /* TODO : remme */ +  /* Confort pour l'insertion de texte */  #define SL(str) str, strlen(str) +/* Espace entre les colonnes */ +#define COL_MARGIN 23 +  /* Accompagnement du dessin pour compléments */  typedef void (* buffer_line_draw_fc) (GBufferLine *, GdkDrawable *, GdkGC *, gint, gint, void *); diff --git a/src/glibext/gcodebuffer.c b/src/glibext/gcodebuffer.c index 573d07d..aec3303 100644 --- a/src/glibext/gcodebuffer.c +++ b/src/glibext/gcodebuffer.c @@ -803,12 +803,12 @@ void g_buffer_view_get_size(GBufferView *view, gint *width, gint *height, bool a      size_t first;                           /* Première ligne intégrée     */      size_t last;                            /* Dernière ligne intégrée     */ -    *width = 0; -    *height = view->line_height; -      if (!WIDTHS_CACHED(view))          g_buffer_view_compute_required_widths(view); +    *width = view->left_text; +    *height = view->line_height; +      for (i = 0; i < BLC_COUNT; i++)      {          if (i == BLC_ADDRESS && !addr) continue; @@ -816,6 +816,9 @@ void g_buffer_view_get_size(GBufferView *view, gint *width, gint *height, bool a          *width += view->max_widths[i]; +        if ((i + 1) < BLC_COUNT) +            *width += COL_MARGIN; +      }      first = g_code_buffer_get_index_from_address(view->buffer, view->start); @@ -1083,7 +1086,7 @@ bool g_buffer_view_get_address_coordinates(GBufferView *view, vmpa_t addr, gint      first = g_code_buffer_get_index_from_address(view->buffer, view->start);      last = g_code_buffer_get_index_from_address(view->buffer, view->end); -    for (i = first; i < last; i++) +    for (i = first; i <= last; i++)      {          current = g_buffer_line_get_address(view->buffer->lines[i]); diff --git a/src/gtkext/Makefile.am b/src/gtkext/Makefile.am index f590693..863c7a8 100644 --- a/src/gtkext/Makefile.am +++ b/src/gtkext/Makefile.am @@ -18,6 +18,9 @@ libgtkext_la_SOURCES =					\  	iodamarshal.h iodamarshal.c			\  	support.h support.c +libgtkext_la_LIBADD = 					\ +	graph/libgtkextgraph.la +  libgtkext_la_LDFLAGS =  @@ -27,6 +30,8 @@ AM_CPPFLAGS =  AM_CFLAGS = $(DEBUG_CFLAGS) $(WARNING_FLAGS) $(COMPLIANCE_FLAGS) +SUBDIRS = graph +  iodamarshal.h: iodamarshal.list  	glib-genmarshal --header $< > $@ diff --git a/src/graph/Makefile.am b/src/gtkext/graph/Makefile.am index e374709..a631a90 100755 --- a/src/graph/Makefile.am +++ b/src/gtkext/graph/Makefile.am @@ -1,12 +1,12 @@ -noinst_LTLIBRARIES  = libgraph.la +noinst_LTLIBRARIES  = libgtkextgraph.la -libgraph_la_SOURCES =					\ +libgtkextgraph_la_SOURCES =				\  	dot.h dot.c							\  	layout.h layout.c					\  	node.h node.c -libgraph_la_LDFLAGS =  +libgtkextgraph_la_LDFLAGS =   INCLUDES = $(LIBGTK_CFLAGS) $(LIBXML_CFLAGS) $(LIBGRAPH_CFLAGS) diff --git a/src/graph/dot.c b/src/gtkext/graph/dot.c index 5a80f59..396aa93 100644 --- a/src/graph/dot.c +++ b/src/gtkext/graph/dot.c @@ -66,10 +66,13 @@ graph_layout *create_graph_layout(char *cmds) +    //printf("CMDS =======\n%s\n\n=================\n", cmds); +      ret = gvLayout(result->context, result->graph, "dot"); +    if (ret != 0) goto cdl_error; -    printf("ret = %d\n", ret); +    //printf("ret = %d\n", ret);      /*      ret = gvLayoutJobs(result->context, result->graph); @@ -78,14 +81,14 @@ graph_layout *create_graph_layout(char *cmds)      printf("ret = %d\n", ret);      */ -      ret = gvRender(result->context, result->graph, "dot", NULL); +    if (ret != 0) goto cdl_error; -    ret = gvRender(result->context, result->graph, "plain", NULL); +    //ret = gvRender(result->context, result->graph, "plain", NULL); -    printf("ret = %d\n", ret); +    //printf("ret = %d\n", ret); @@ -148,7 +151,7 @@ void place_nodes_of_graph_layout(const graph_layout *layout, GtkGraphView *view,  {      int height;                             /* Hauteur du graphique        */      node_t *iter;                           /* Boucle de parcours          */ -    const GGraphNode *node;                 /* Intermédiaire concerné      */ +    GGraphNode *node;                       /* Intermédiaire concerné      */      height = GD_bb(layout->graph).UR.y; @@ -165,6 +168,8 @@ void place_nodes_of_graph_layout(const graph_layout *layout, GtkGraphView *view,  *                                                                             *  *  Paramètres  : layout = graphique à supprimer de la mémoire.                *  *                count  = nombre d'éléments mis en place.                     * +*                nodes  = liste de noeuds à consulter.                        * +*                ncount = taille de la liste des noeuds.                      *  *                                                                             *  *  Description : Charge la définition de tous les liens graphiques.           *  *                                                                             * @@ -174,17 +179,21 @@ void place_nodes_of_graph_layout(const graph_layout *layout, GtkGraphView *view,  *                                                                             *  ******************************************************************************/ -GtkLinkRenderer **create_links_from_graph_layout(const graph_layout *layout, size_t *count) +GtkLinkRenderer **create_links_from_graph_layout(const graph_layout *layout, size_t *count, GGraphNode **nodes, size_t ncount)  {      GtkLinkRenderer **result;               /* Liste à retourner           */      int height;                             /* Hauteur du graphique        */      node_t *niter;                          /* Boucle de parcours #1       */      edge_t *eiter;                          /* Boucle de parcours #2       */ +    GdkPoint *points;                       /* Points de ligne relus       */ +    size_t points_count;                    /* Nombre de ces points        */      splines *lines;                         /* Lignes déjà tracées         */ +    GGraphNode *node;                       /* Noeud rattaché              */      int i;                                  /* Boucle de parcours #3       */      int k;                                  /* Boucle de parcours #4       */ -    GdkPoint *points;                       /* Points de ligne relus       */ -    size_t points_count;                    /* Nombre de ces points        */ +    bezier *bez;                            /* Courbe à reproduire         */ +    Agsym_t *attrib;                        /* Couleur d'un lien           */ +    LinkColor color;                        /* Couleur d'impression        */      result = NULL;      *count = 0; @@ -194,37 +203,62 @@ GtkLinkRenderer **create_links_from_graph_layout(const graph_layout *layout, siz      for (niter = agfstnode(layout->graph); niter != NULL; niter = agnxtnode(layout->graph, niter))          for (eiter = agfstout(layout->graph, niter); eiter != NULL; eiter = agnxtout(layout->graph, eiter))          { +            points = NULL; +            points_count = 0; +              lines = ED_spl(eiter); -            //printf("edge == %p\n", eiter); +            /* Raccordement au point de départ */ -            points = NULL; -            points_count = 0; +            node = find_graph_node_by_name(nodes, ncount, agtail(eiter)->name); + +            g_graph_node_connect(node, +                                 lines->list[0].list[0].x, +                                 height - lines->list[0].list[0].y, +                                 &points, &points_count); + +            /* Tracé du lien... */              for (i = 0; i < lines->size; i++)              { -                points = (GdkPoint *)realloc(points, (points_count + lines->list[i].size) * sizeof(GdkPoint)); +                bez = &lines->list[i]; -                for (k = 0; k < lines->list[i].size; k++) +                points = (GdkPoint *)realloc(points, (points_count + bez->size) * sizeof(GdkPoint)); + +                for (k = 0; k < bez->size; k++)                  { -                    points[points_count + k].x = lines->list[i].list[k].x; -                    points[points_count + k].y = height - lines->list[i].list[k].y; +                    points[points_count + k].x = bez->list[k].x; +                    points[points_count + k].y = height - bez->list[k].y; +                } -                    /* -                    printf("  ... ( %d ; %d)\n", -                           lines->list[i].list[k].x, height - lines->list[i].list[k].y); -                    */ +                points_count += bez->size; -                } +            } -                points_count += lines->list[i].size; +            /* Raccordement au point d'arrivée */ -                //printf("  ...\n"); +            node = find_graph_node_by_name(nodes, ncount, aghead(eiter)->name); -            } +            g_graph_node_connect(node, +                                 bez->list[k - 1].x, +                                 height - bez->list[k - 1].y, +                                 &points, &points_count); + +            /* Détermination de la couleur */ + +            attrib = agfindedgeattr(agraphof(agtail(eiter)), "color");              result = (GtkLinkRenderer **)realloc(result, ++(*count) * sizeof(GtkLinkRenderer *)); -            result[*count - 1] = GTK_LINK_RENDERER(gtk_link_renderer_new(points, points_count)); + +            if (eiter->attr[attrib->index][0] == 'g')   /* "green" */ +                color = LKC_GREEN; +            else if (eiter->attr[attrib->index][0] == 'r')   /* "red" */ +                color = LKC_RED; +            else +                color = LKC_DEFAULT; + +            result[*count - 1] = GTK_LINK_RENDERER(gtk_link_renderer_new(color, +                                                                         points, points_count));          } diff --git a/src/graph/dot.h b/src/gtkext/graph/dot.h index 47550a8..d610e1b 100644 --- a/src/graph/dot.h +++ b/src/gtkext/graph/dot.h @@ -26,7 +26,7 @@  #include "node.h" -#include "../gtkext/gtklinkrenderer.h" +#include "../gtklinkrenderer.h" @@ -44,7 +44,7 @@ void delete_graph_layout(graph_layout *);  void place_nodes_of_graph_layout(const graph_layout *, GtkGraphView *, GGraphNode **, size_t);  /* Charge la définition de tous les liens graphiques. */ -GtkLinkRenderer **create_links_from_graph_layout(const graph_layout *, size_t *); +GtkLinkRenderer **create_links_from_graph_layout(const graph_layout *, size_t *, GGraphNode **, size_t); diff --git a/src/graph/layout.c b/src/gtkext/graph/layout.c index f8afad6..69e32cc 100644 --- a/src/graph/layout.c +++ b/src/gtkext/graph/layout.c @@ -31,9 +31,9 @@  #include "dot.h"  #include "node.h" -#include "../analysis/binary.h" -#include "../common/extstr.h" -#include "../gtkext/gtkbufferview.h" +#include "../gtkbufferview.h" +#include "../../analysis/binary.h" +#include "../../common/extstr.h" @@ -78,7 +78,7 @@ bool build_graph_view(GtkGraphView *view, GtkViewPanel **views, size_t count)      /* Définition du graphique */ -    cmds = strdup("digraph G {\noverlap=false;\n  splines=true;\n"); +    cmds = strdup("digraph G {\noverlap=false;\n  splines=ortho;\n  compound=true;\n");      for (i = 0; i < count; i++)          cmds = g_graph_node_register_for_dot(nodes[i], cmds); @@ -93,7 +93,7 @@ bool build_graph_view(GtkGraphView *view, GtkViewPanel **views, size_t count)      place_nodes_of_graph_layout(layout, view,  nodes, count); -    links = create_links_from_graph_layout(layout, &links_count); +    links = create_links_from_graph_layout(layout, &links_count,  nodes, count);      gtk_graph_view_attach_links(view, links, links_count);      gtk_widget_queue_draw(GTK_WIDGET(view)); @@ -169,17 +169,25 @@ static char *complete_graph_links(const GtkGraphView *view, GtkViewPanel **views                      switch (types[j])                      {                          case ILT_JUMP: -                            snprintf(cmd, LINKS_DESC_LEN, "_%p -> _%p;\n", views[i], views[k]); +                            snprintf(cmd, LINKS_DESC_LEN, +                                     "_%p:s -> _%p:n  [ltail=cluster_%p, lhead=cluster_%p];\n", +                                     views[i], views[k], views[i], views[k]);                              desc = stradd(desc, cmd);                              break;                          case ILT_JUMP_IF_TRUE: -                            snprintf(cmd, LINKS_DESC_LEN, "_%p -> _%p;\n", views[i], views[k]); +                            snprintf(cmd, LINKS_DESC_LEN, +                                     "_%p:s -> _%p:n  [ltail=cluster_%p, lhead=cluster_%p, " \ +                                     "color=green];\n", +                                     views[i], views[k], views[i], views[k]);                              desc = stradd(desc, cmd);                              break;                          case ILT_JUMP_IF_FALSE: -                            snprintf(cmd, LINKS_DESC_LEN, "_%p -> _%p;\n", views[i], views[k]); +                            snprintf(cmd, LINKS_DESC_LEN, +                                     "_%p:s -> _%p:n  [ltail=cluster_%p, lhead=cluster_%p, " \ +                                     "color=red];\n", +                                     views[i], views[k], views[i], views[k]);                              desc = stradd(desc, cmd);                              break; @@ -192,10 +200,10 @@ static char *complete_graph_links(const GtkGraphView *view, GtkViewPanel **views          } -        /* Si la ligne n'est pas la dernière, on suit le flux normal */ -        else if (addr != end) +        /* Sinon on suit le flux normal */ +        else          { -            next = g_arch_instruction_get_next_iter(instrs, last, end); +            next = g_arch_instruction_get_next_iter(instrs, last, VMPA_MAX);              if (next == NULL) continue;              g_arch_instruction_get_location(next, NULL, NULL, &addr); @@ -206,7 +214,9 @@ static char *complete_graph_links(const GtkGraphView *view, GtkViewPanel **views              if (k < count)              { -                snprintf(cmd, LINKS_DESC_LEN, "_%p -> _%p;\n", views[i], views[k]); +                snprintf(cmd, LINKS_DESC_LEN, +                         "_%p:s -> _%p:n  [ltail=cluster_%p, lhead=cluster_%p];\n", +                         views[i], views[k], views[i], views[k]);                  desc = stradd(desc, cmd);              } diff --git a/src/graph/layout.h b/src/gtkext/graph/layout.h index 006fa9b..ffb227f 100644 --- a/src/graph/layout.h +++ b/src/gtkext/graph/layout.h @@ -28,8 +28,8 @@  #include <stdbool.h> -#include "../gtkext/gtkgraphview.h" -#include "../gtkext/gtkviewpanel.h" +#include "../gtkgraphview.h" +#include "../gtkviewpanel.h" diff --git a/src/graph/node.c b/src/gtkext/graph/node.c index 0d278f3..f3e6352 100644 --- a/src/graph/node.c +++ b/src/gtkext/graph/node.c @@ -29,7 +29,7 @@  #include <string.h> -#include "../common/extstr.h" +#include "../../common/extstr.h" @@ -44,6 +44,8 @@ struct _GGraphNode      GtkWidget *view;                        /* Morceau de code représenté  */      char *name;                             /* Adresse sous forme humaine  */ +    GtkAllocation alloc;                    /* Emplacement du bloc rattaché*/ +  }; @@ -200,6 +202,12 @@ char *g_graph_node_register_for_dot(const GGraphNode *node, char *cmds)      gtk_widget_size_request(node->view, &requisition); +    snprintf(buffer, 128, "  subgraph cluster%s {\n", node->name); +    cmds = stradd(cmds, buffer); + +    cmds = stradd(cmds, "    style=invisible;\n"); + +    cmds = stradd(cmds, "    ");      cmds = stradd(cmds, node->name);      cmds = stradd(cmds, " [shape=box, fixedsize "); @@ -213,6 +221,8 @@ char *g_graph_node_register_for_dot(const GGraphNode *node, char *cmds)      cmds = stradd(cmds, "];\n"); +    cmds = stradd(cmds, "  }\n"); +      return cmds;  } @@ -233,7 +243,7 @@ char *g_graph_node_register_for_dot(const GGraphNode *node, char *cmds)  *                                                                             *  ******************************************************************************/ -void g_graph_node_place(const GGraphNode *node, GtkGraphView *view, gint x, gint y) +void g_graph_node_place(GGraphNode *node, GtkGraphView *view, gint x, gint y)  {      GtkRequisition requisition;             /* Taille à l'écran actuelle   */ @@ -244,6 +254,66 @@ void g_graph_node_place(const GGraphNode *node, GtkGraphView *view, gint x, gint      gtk_graph_view_put(view, node->view, x, y); +    node->alloc.x = x; +    node->alloc.y = y; +    node->alloc.width = requisition.width; +    node->alloc.height = requisition.height; + +} + + +/****************************************************************************** +*                                                                             * +*  Paramètres  : node   = intermédiaire à consulter.                          * +*                x      = abscisse du point à relier.                         * +*                y      = ordonnée du point à relier.                         * +*                points = liste de points à mettre à jour. [OUT]              * +*                count  = taille de cette même liste. [OUT]                   * +*                                                                             * +*  Description : Etablit une jonction ferme avec un noeud.                    * +*                                                                             * +*  Retour      : -                                                            * +*                                                                             * +*  Remarques   : -                                                            * +*                                                                             * +******************************************************************************/ + +void g_graph_node_connect(const GGraphNode *node, gint x, gint y, GdkPoint **points, size_t *count) +{ +    const GtkAllocation *alloc;             /* Emplacement du bloc rattaché*/ + +    alloc = &node->alloc; + +    *points = (GdkPoint *)realloc(*points, ++(*count) * sizeof(GdkPoint)); + +    /* Si le point est sur la gauche... */ +    if (alloc->y <= y && y < (alloc->y + alloc->height) && x < alloc->x) +    { +        (*points)[*count - 1].x = alloc->x; +        (*points)[*count - 1].y = y; +    } + +    /* Si le point est sur la droite... */ +    else if (alloc->y <= y && y < (alloc->y + alloc->height) && x > (alloc->x + alloc->width)) +    { +        (*points)[*count - 1].x = alloc->x + alloc->width; +        (*points)[*count - 1].y = y; +    } + +    /* Si le point est au dessus... */ +    else if (alloc->x <= x && x < (alloc->x + alloc->width) && y < alloc->y) +    { +        (*points)[*count - 1].x = x; +        (*points)[*count - 1].y = alloc->y; +    } + +    /* Si le point est en dessous... */ +    else if (alloc->x <= x && x < (alloc->x + alloc->width) && y > (alloc->y + alloc->height)) +    { +        (*points)[*count - 1].x = x; +        (*points)[*count - 1].y = alloc->y + alloc->height; +    } +  } @@ -267,9 +337,9 @@ void g_graph_node_place(const GGraphNode *node, GtkGraphView *view, gint x, gint  *                                                                             *  ******************************************************************************/ -const GGraphNode *find_graph_node_by_name(GGraphNode **nodes, size_t count, const char *target) +GGraphNode *find_graph_node_by_name(GGraphNode **nodes, size_t count, const char *target)  { -    const GGraphNode *result;               /* Trouvaille à remonter       */ +    GGraphNode *result;                     /* Trouvaille à remonter       */      size_t i;                               /* Boucle de parcours          */      result = NULL; diff --git a/src/graph/node.h b/src/gtkext/graph/node.h index f8d0078..515c92b 100644 --- a/src/graph/node.h +++ b/src/gtkext/graph/node.h @@ -25,7 +25,7 @@  #define _GRAPH_NODE_H -#include "../gtkext/gtkgraphview.h" +#include "../gtkgraphview.h" @@ -56,7 +56,10 @@ GGraphNode *g_graph_node_new(GtkWidget *);  char *g_graph_node_register_for_dot(const GGraphNode *, char *);  /* Place le morceau de code de l'intermédiaire à l'écran. */ -void g_graph_node_place(const GGraphNode *, GtkGraphView *, gint , gint); +void g_graph_node_place(GGraphNode *, GtkGraphView *, gint , gint); + +/* Etablit une jonction ferme avec un noeud. */ +void g_graph_node_connect(const GGraphNode *, gint, gint, GdkPoint **, size_t *); @@ -64,7 +67,7 @@ void g_graph_node_place(const GGraphNode *, GtkGraphView *, gint , gint);  /* Recherche un noeud donné dans une série de noeuds. */ -const GGraphNode *find_graph_node_by_name(GGraphNode **, size_t, const char *); +GGraphNode *find_graph_node_by_name(GGraphNode **, size_t, const char *); diff --git a/src/gtkext/gtkgraphview.c b/src/gtkext/gtkgraphview.c index 9fa9135..c0ea347 100644 --- a/src/gtkext/gtkgraphview.c +++ b/src/gtkext/gtkgraphview.c @@ -27,7 +27,7 @@  #include "gtkblockview.h"  #include "gtkbufferview.h"  #include "gtkviewpanel-int.h" -#include "../graph/layout.h" +#include "graph/layout.h" @@ -554,8 +554,8 @@ static GtkViewPanel **gtk_graph_view_load_nodes(GtkGraphView *view, GLoadedBinar      list = g_loaded_binary_get_instructions(binary);      buffer = g_loaded_binary_get_disassembled_buffer(binary); -    addr = g_loaded_binary_display_addresses_in_text(binary); -    code = g_loaded_binary_display_code_in_text(binary); +    addr = g_loaded_binary_display_addresses_in_text(binary, BDT_GRAPH); +    code = g_loaded_binary_display_code_in_text(binary, BDT_GRAPH);      first = start;      last = first; @@ -564,7 +564,7 @@ static GtkViewPanel **gtk_graph_view_load_nodes(GtkGraphView *view, GLoadedBinar           iter != NULL;           iter = g_arch_instruction_get_next_iter(list, iter, end))      { -        if (g_arch_instruction_has_sources(iter)) +        if (first != VMPA_MAX && g_arch_instruction_has_sources(iter))          {              result = (GtkViewPanel **)realloc(result, ++(*count) * sizeof(GtkViewPanel *)); diff --git a/src/gtkext/gtklinkrenderer.c b/src/gtkext/gtklinkrenderer.c index 9332860..2913c19 100644 --- a/src/gtkext/gtklinkrenderer.c +++ b/src/gtkext/gtklinkrenderer.c @@ -24,12 +24,16 @@  #include "gtklinkrenderer.h" +#include <math.h> + +  /* Lien entre morceaux de code (instance) */  struct _GtkLinkRenderer  {      GtkObject parent;                       /* A laisser en premier        */ +    LinkColor color;                        /* Couleur d'impression        */      GdkPoint *points;                       /* Points de la ligne dessinée */      size_t count;                           /* Quantité de ces points      */ @@ -44,12 +48,18 @@ struct _GtkLinkRendererClass  }; +#define ARROW_LENGHT 10 +#define ARROW_DEGREES 10 + +  /* Initialise la classe générique des liens graphiques. */  static void gtk_link_renderer_class_init(GtkLinkRendererClass *);  /* Initialise une instance de lien graphique entre codes. */  static void gtk_link_renderer_init(GtkLinkRenderer *); +/* Dessine une flèche au bout du lien représenté. */ +static void draw_link_arrow(cairo_t *, gint, gint, gint, gint); @@ -95,7 +105,8 @@ static void gtk_link_renderer_init(GtkLinkRenderer *view)  /******************************************************************************  *                                                                             * -*  Paramètres  : points = points consituant la ligne à représenter.           * +*  Paramètres  : color  = couleur d'impression.                               * +ù                points = points consituant la ligne à représenter.           *  *                count  = nombre de ces points.                               *  *                                                                             *  *  Description : Crée un nouveau moteur de rendu pour les liens graphiques.   * @@ -106,12 +117,13 @@ static void gtk_link_renderer_init(GtkLinkRenderer *view)  *                                                                             *  ******************************************************************************/ -GtkObject *gtk_link_renderer_new(GdkPoint *points, size_t count) +GtkObject *gtk_link_renderer_new(LinkColor color, GdkPoint *points, size_t count)  {      GtkLinkRenderer *result;                /* Moteur de rendu à retourner */      result = g_object_new(GTK_TYPE_LINK_RENDERER, NULL); +    result->color = color;      result->points = points;      result->count = count; @@ -136,6 +148,88 @@ GtkObject *gtk_link_renderer_new(GdkPoint *points, size_t count)  void gtk_link_renderer_draw(const GtkLinkRenderer *renderer, GdkDrawable *drawable, GdkGC *gc)  { -    gdk_draw_lines(drawable, gc, renderer->points, renderer->count); +    cairo_t *cairo;                         /* Gestionnaire de rendu       */ +    size_t i;                               /* Boucle de parcours          */ + +    cairo = gdk_cairo_create(drawable); + +    switch (renderer->color) +    { +        default: +        case LKC_DEFAULT: +            cairo_set_source_rgb(cairo, 0, 0, 0); +            break; +        case LKC_GREEN: +            cairo_set_source_rgb(cairo, 0, 0.6, 0); +            break; +        case LKC_RED: +            cairo_set_source_rgb(cairo, 0.8, 0, 0); +            break; +    } + +    cairo_set_line_width(cairo, 1); + +    cairo_move_to(cairo, renderer->points[0].x, renderer->points[0].y); + +    for (i = 1; i < renderer->count; i++) +        cairo_line_to(cairo, renderer->points[i].x, renderer->points[i].y); + +    cairo_stroke(cairo); + +    draw_link_arrow(cairo, +                    renderer->points[renderer->count - 2].x, +                    renderer->points[renderer->count - 2].y, +                    renderer->points[renderer->count - 1].x, +                    renderer->points[renderer->count - 1].y); + +    cairo_destroy(cairo); + +} + + +/****************************************************************************** +*                                                                             * +*  Paramètres  : cairo   = gestionnaire de rendu graphique.                   * +*                start_x = abscisse du début du segment final.                * +*                start_y = ordonnée du début du segment final.                * +*                end_x   = abscisse de fin de segment et pointe de flèche.    * +*                end_y   = ordonnée de fin de segment et pointe de flèche.    * +*                                                                             * +*  Description : Dessine une flèche au bout du lien représenté.               * +*                                                                             * +*  Retour      : -                                                            * +*                                                                             * +*  Remarques   : -                                                            * +*                                                                             * +******************************************************************************/ + +static void draw_link_arrow(cairo_t *cairo, gint start_x, gint start_y, gint end_x, gint end_y) +{ +    double angle;                           /* Angle du triangle à remplir */ +    double factor_x;                        /* Direction de la flèche #1   */ +    double factor_y;                        /* Direction de la flèche #2   */ +    double x1;                              /* Abscisse du point n°1       */ +    double y1;                              /* Ordonnée du point n°1       */ +    double x2;                              /* Abscisse du point n°2       */ +    double y2;                              /* Ordonnée du point n°2       */ + +    angle = atan2(end_y - start_y, end_x - start_x) + M_PI; + +    factor_x = -1; +    factor_y = -1; + +    x1 = end_x + factor_x * ARROW_LENGHT * cos(angle - ARROW_DEGREES); +    y1 = end_y + factor_y * ARROW_LENGHT * sin(angle - ARROW_DEGREES); +    x2 = end_x + factor_x * ARROW_LENGHT * cos(angle + ARROW_DEGREES); +    y2 = end_y + factor_y * ARROW_LENGHT * sin(angle + ARROW_DEGREES); + +    cairo_move_to(cairo, end_x, end_y); + +    cairo_line_to(cairo, x1, y1); +    cairo_line_to(cairo, x2, y2); + +    cairo_move_to(cairo, end_x, end_y); + +    cairo_fill(cairo);  } diff --git a/src/gtkext/gtklinkrenderer.h b/src/gtkext/gtklinkrenderer.h index d662454..95cdea6 100644 --- a/src/gtkext/gtklinkrenderer.h +++ b/src/gtkext/gtklinkrenderer.h @@ -45,11 +45,23 @@ typedef struct _GtkLinkRenderer GtkLinkRenderer;  typedef struct _GtkLinkRendererClass GtkLinkRendererClass; +/* Couleur de représentation */ +typedef enum _LinkColor +{ +    LKC_DEFAULT,                            /* Noir, par défaut            */ +    LKC_GREEN,                              /* Condition vérifiée          */ +    LKC_RED,                                /* Condition non vérifiée      */ + +    LKC_COUNT + +} LinkColor; + +  /* Détermine le type du moteur de rendu pour les liens graphiques. */  GType gtk_link_renderer_get_type(void);  /* Crée un nouveau moteur de rendu pour les liens graphiques. */ -GtkObject *gtk_link_renderer_new(GdkPoint *, size_t); +GtkObject *gtk_link_renderer_new(LinkColor, GdkPoint *, size_t);  /* Dessine les liens graphiques enregistrés dans le moteur. */  void gtk_link_renderer_draw(const GtkLinkRenderer *, GdkDrawable *, GdkGC *); diff --git a/src/gtkext/gtkviewpanel.c b/src/gtkext/gtkviewpanel.c index 3ccc0ff..0ab4f72 100644 --- a/src/gtkext/gtkviewpanel.c +++ b/src/gtkext/gtkviewpanel.c @@ -293,8 +293,7 @@ static gboolean gtk_view_panel_expose(GtkWidget *widget, GdkEventExpose *event)      GtkViewPanel *panel;      GdkGCValues values;                     /* Propriétés du contexte      */      GtkStyle *style;                        /* Style associé au composant  */ -    int width;                              /* Largeur de l'élément        */ -    int height;                             /* Hauteur de l'élément        */ +    GtkRequisition req;                     /* Taille allouée à l'élément  */      GtkStateType state;                     /* Etat du composant           */      panel = GTK_VIEW_PANEL(widget); @@ -304,13 +303,13 @@ static gboolean gtk_view_panel_expose(GtkWidget *widget, GdkEventExpose *event)          gdk_gc_get_values(panel->gc, &values);          style = gtk_widget_get_style(widget); -        gtk_widget_get_size_request(widget, &width, &height); +        gtk_widget_size_request(widget, &req);          state = gtk_widget_get_state(widget);          gdk_gc_set_foreground(panel->gc, &style->dark[state]);          gdk_draw_rectangle(GDK_DRAWABLE(widget->window), panel->gc, -                           FALSE, 0, 0, width - 1, height - 1); +                           FALSE, 0, 0, req.width - 1, req.height - 1);          gdk_gc_set_foreground(panel->gc, &values.foreground); diff --git a/src/project.c b/src/project.c index b6526c7..106bf3a 100644 --- a/src/project.c +++ b/src/project.c @@ -390,8 +390,8 @@ size_t g_study_project_attach_binary(GStudyProject *project, GLoadedBinary *bina          loaded->views[i] = GTK_VIEW_PANEL(view);          gtk_view_panel_attach_binary(loaded->views[i], binary, -                                     g_loaded_binary_display_addresses_in_text(binary), -                                     g_loaded_binary_display_code_in_text(binary)); +                                     g_loaded_binary_display_addresses_in_text(binary, BDT_ASM), +                                     g_loaded_binary_display_code_in_text(binary, BDT_ASM));          /* Intégration finale dans un support défilant */ | 
