]> git.siccegge.de Git - frida/frida.git/blobdiff - src/gui/widgets/BasicBlockWidget.cxx
Fix formating of comments. Should not start with a linebreak
[frida/frida.git] / src / gui / widgets / BasicBlockWidget.cxx
index 37814014596b29e860a53778b9b09cfbc551ff33..2a3f3b7d992f30f4f5089754a1016bf221ac13c7 100644 (file)
@@ -1,30 +1,47 @@
 #include "BasicBlockWidget.hxx"
+#include "CFGScene.hxx"
 #include "gui/Mainwindow.hxx"
 #include "gui/dialogs/SimpleStringDialog.hxx"
 #include "core/BasicBlock.hxx"
 #include "core/Function.hxx"
+#include "core/Comment.hxx"
+#include "disassembler/Instruction.hxx"
 #include "core/InformationManager.hxx"
 #include "core/events/RenameFunctionEvent.hxx"
-#include <iostream>
+#include "core/events/ChangeCommentEvent.hxx"
+#include <algorithm>
 
 class CustomQGraphicsTextItem : public QObject, public QGraphicsTextItem {
 public:
        CustomQGraphicsTextItem(const QString& text, BasicBlockWidget* parent)
                : QGraphicsTextItem(text, parent), parent(parent) {}
        void contextMenuEvent(QGraphicsSceneContextMenuEvent*);
+
+       void adjustSize();
 private:
-       void addComment(QTextTableCell cell, const QString& title);
+       void addComment(int row, bool global);
 
        BasicBlockWidget* parent;
 };
 
-void CustomQGraphicsTextItem::addComment(QTextTableCell cell, const QString& title) {
-       SimpleStringDialog dialog(title);
+void CustomQGraphicsTextItem::addComment(int row, bool global) {
+       SimpleStringDialog dialog(global ? "Global comment" : "Local comment");
        int result = dialog.exec();
+       uint64_t address = parent->instructions[row].getAddress();
        if (QDialog::Accepted == result) {
-               LOG4CXX_DEBUG(parent->logger, "adding comment " << dialog.result().toStdString()
-                             << " at row " << cell.row());
-               cell.firstCursorPosition().insertHtml(QString(";; ") + dialog.result());
+               Comment* comment;
+               if (global) {
+                       comment = parent->block->getManager()->newGlobalComment(address);
+               } else {
+                       /* TODO: 0x23 as we currently don't have the function here
+                        * and setting it to null will make the comment appear
+                        * global. Also means local comments are largely still
+                        * broken.
+                        */
+                       comment = parent->block->getManager()->newLocalComment(address, (Function*)0x23);
+               }
+               comment->setText(dialog.result().toStdString());
+               parent->block->getManager()->finishComment(comment);
        } else {
                LOG4CXX_DEBUG(parent->logger, "addComment aborted");
        }
@@ -38,32 +55,64 @@ void CustomQGraphicsTextItem::contextMenuEvent(QGraphicsSceneContextMenuEvent* e
        QMenu menu;
        bool ok;
        uint64_t address = c.selectedText().toLongLong(&ok, 16);
+       QTextTable* table = c.currentTable();
        if (ok) {
                QAction* act = menu.addAction(c.selectedText() + " is a Function");
                QObject::connect(act, &QAction::triggered,
-                                [=]() {parent->mainwindow->requestNewFunctionByAddress(address);});
+                                [=]() {
+                                        parent->mainwindow->requestNewFunctionByAddress(address);
+                                        if (NULL == table) return;
+                                        int row = table->cellAt(c).row();
+                                        uint64_t insAddress = parent->instructions[row].getAddress();
+                                        Comment* comment = parent->block->getManager()->newLocalComment(insAddress, (Function*)0x23);
+                                        comment->setText("#F<" + c.selectedText().toStdString() + ">");
+                                        parent->block->getManager()->finishComment(comment);
+                                });
        }
 
-       QTextTable* table = c.currentTable();
        if (NULL != table) {
                int row = table->cellAt(c).row();
-               QTextTableCell cell = table->cellAt(row, 2);
                QAction* globalComment = menu.addAction("Add global Comment");
                QAction* localComment = menu.addAction("Add local Comment");
 
                QObject::connect(globalComment, &QAction::triggered,
-                                [=]() { addComment(cell, "Global comment"); });
+                                [=]() { addComment(row, true); });
                QObject::connect(localComment, &QAction::triggered,
-                                [=]() { addComment(cell, "Local comment"); });
+                                [=]() { addComment(row, false); });
        }
 
        menu.exec(event->screenPos());
 }
 
+/* QGraphicsTextItem has an adjustSize() function that is supposed to
+ * resize the widget to it's "ideal" size. However it totally ignores
+ * all directives to not wrap lines and "ideal" is actually just a
+ * bunch of heuristics.
+ *
+ * We are starting with a hopefully absurdly large startingwidth and
+ * reduce it untill a line is broken (detected by a change in
+ * height). As long as the width (1000 here) is sufficiently large,
+ * this should give us a widget without any line-wrapping.
+ *
+ * One needs to call this on a Pointer of tye CustomQGraphicsTextItem
+ * as the adjustSize() function is not polymorphic (vurtual).
+ */
+void CustomQGraphicsTextItem::adjustSize() {
+       int width = 1000;
+       setTextWidth(width);
+       int height = boundingRect().height();
+       while (width > 250 && height == boundingRect().height()) {
+               setTextWidth(width -= 10);
+       }
+       width += 10;
+       if (width < 250) width = 250;
+       setTextWidth(width);
+}
+
 BasicBlockWidget::BasicBlockWidget(const QString& name, BasicBlock * block,
                                    Mainwindow * mainwindow)
-       : width(270), height(45), name(name)
-       , _table(NULL)
+       : width(200), height(45), name(name)
+       , currentColor(defaultColor), _table(NULL)
        , block(block), mainwindow(mainwindow)
        , logger(log4cxx::Logger::getLogger("gui.BasicBlockWidget." + name.toStdString())) {
        next[0] = NULL; next[1] = NULL;
@@ -82,10 +131,26 @@ BasicBlockWidget::BasicBlockWidget(const QString& name, BasicBlock * block,
                                 if (str.startsWith("function:")) {
                                         QString address = str.remove("function:");
                                         mainwindow->switchMainPlaneToAddress(address.toInt(NULL, 16));
+                                } else if (str.startsWith("block:")) {
+                                        QString address = str.remove("block:");
+
+                                        /* next[0] is always the jumptarget. On a
+                                         * conditional jump, next[1] also
+                                         * contains the following instruction
+                                         *
+                                         * TODO: Verify we're switching to the
+                                         *       right block -- the target
+                                         *       address matches the next blocks
+                                         *       start address
+                                         */
+                                        LOG4CXX_TRACE(logger, "Highlighting block at Address " << address.toStdString()
+                                                      << " BasicBlockWidget " << std::hex << next[0]);
+                                        ((CFGScene*)this->scene())->highlightBlock(next[0]);
                                 }
                         });
        instructions = block->getInstructions();
        populateWidget();
+       block->getManager()->registerChangeCommentEvent([=](ChangeCommentEvent* e) {changeCommentHandler(e);});
 }
 
 void BasicBlockWidget::updateFunctionName(RenameFunctionEvent *event) {
@@ -99,13 +164,24 @@ void BasicBlockWidget::updateFunctionName(RenameFunctionEvent *event) {
                        if (isLink)
                        {
                                if (search == format.anchorHref()) {
-                                       LOG4CXX_DEBUG(logger, i.fragment().text().toStdString() << " ---> " << format.anchorHref().toStdString());
+                                       LOG4CXX_DEBUG(logger, i.fragment().text().toStdString() << " ---> "
+                                                     << format.anchorHref().toStdString());
+
+                                       /* This should select the function name. It stars
+                                        * by selecting the whole link fragment from back
+                                        * to front and then moves one word to the back
+                                        * again deselecting whatever mnemonic is used for
+                                        * the call instruction.
+                                        */
                                        QTextCursor c(b);
                                        c.setPosition(i.fragment().position());
                                        c.movePosition(QTextCursor::Right, QTextCursor::MoveAnchor, i.fragment().length());
                                        c.movePosition(QTextCursor::Left,  QTextCursor::KeepAnchor, i.fragment().length());
                                        c.movePosition(QTextCursor::WordRight,  QTextCursor::KeepAnchor);
                                        c.insertText(event->new_name.c_str());
+
+                                       QGraphicsTextItem* item = _widget.get();
+                                       ((CustomQGraphicsTextItem*)item)->adjustSize();
                                }
                        }
                }
@@ -113,6 +189,31 @@ void BasicBlockWidget::updateFunctionName(RenameFunctionEvent *event) {
        }
 }
 
+void BasicBlockWidget::changeCommentHandler(ChangeCommentEvent* event) {
+       auto inst_it = std::find_if(instructions.begin(), instructions.end(),
+                                   [=](Instruction& inst) {
+                                           return inst.getAddress() == event->address;
+                                   });
+       if (inst_it != instructions.end()) {
+               if (std::find(inst_it->comments().begin(),
+                             inst_it->comments().begin(),
+                             event->comment) == inst_it->comments().end()) {
+                       LOG4CXX_DEBUG(logger, "Change Comment Event --  New Comment!");
+                       inst_it->comments().push_back(event->comment);
+               }
+               int row = inst_it - instructions.begin();
+               LOG4CXX_DEBUG(logger, "Inserting comment for instruction at row " << std::hex << row);
+               QTextCursor cursor = _table->cellAt(row, 2).lastCursorPosition();
+               while (cursor != _table->cellAt(row, 2).firstCursorPosition()) {
+                       cursor.movePosition(QTextCursor::Left, QTextCursor::KeepAnchor, 1);
+               }
+               cursor.removeSelectedText();
+               cursor.insertHtml(formatComments(&*inst_it));
+               QGraphicsTextItem* item = _widget.get();
+               ((CustomQGraphicsTextItem*)item)->adjustSize();
+       }
+}
+
 void BasicBlockWidget::populateWidget() {
        int row;
        QTextTableFormat format;
@@ -134,7 +235,7 @@ void BasicBlockWidget::populateWidget() {
                        bytestring += hexdigits[byte & 0xF];
                        bytestring += ' ';
                }
-               _table->cellAt(row, 0).firstCursorPosition().insertText(bytestring);
+               _table->cellAt(row, 0).firstCursorPosition().insertHtml("<nobr>" + bytestring + "</nobr>");
 
                QString line = inst.getText().c_str();
                line = line.replace('\t', ' ').toHtmlEscaped();
@@ -152,19 +253,28 @@ void BasicBlockWidget::populateWidget() {
                        }
                        line = "<a href=\"" + href + "\">" + line + "</a>";
                }
+               _table->cellAt(row, 1).firstCursorPosition().insertHtml("<nobr>" + line + "</nobr>");
+               _table->cellAt(row, 2).firstCursorPosition().insertHtml(formatComments(&inst));
+       }
+       QGraphicsTextItem* item = _widget.get();
+       ((CustomQGraphicsTextItem*)item)->adjustSize();
+}
 
-               _table->cellAt(row, 1).firstCursorPosition().insertHtml(line);
+QString BasicBlockWidget::formatComments(Instruction* inst) {
+       QStringList comments;
+       for (Comment* c: inst->comments()) {
+               comments << QString(c->getText().c_str()).toHtmlEscaped();
        }
+       return (comments.empty() ? "" : ";; ") + comments.join("<br />").trimmed();
 }
 
 void BasicBlockWidget::paint(QPainter *painter, const QStyleOptionGraphicsItem*,
                   QWidget*) {
-       _widget->adjustSize();
        width = 10 + _widget->boundingRect().width();
        height = 25 + _widget->boundingRect().height();
        if (width < 250) width = 250;
 
-       painter->fillRect(0, 0, width, height, QColor(0xcc, 0xcc, 0xff, 0xff));
+       painter->fillRect(0, 0, width, height, currentColor);
        painter->setPen(QColor(0x00, 0x00, 0xff, 0xff));
        painter->drawRect(0, 0, width, height);
        painter->drawText(5, 15, name);