rizinorg/cutter

Comments window should show unescaped comments whenever possible

Ouverte

#2 961 ouverte le 7 juin 2022

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C++ (1 123 forks)batch import
good first issue

Métriques du dépôt

Stars
 (14 852 étoiles)
Métriques de merge PR
 (Merge moyen 13j 23h) (20 PRs mergées en 30 j)

Description

See the comment at 0x1400022c6 address, it's a Chinese words 好的, which are shown in the disassembly but neither in Cutter nor in the Rizin's CCl output:

image

Originally posted by @XVilka in https://github.com/rizinorg/cutter/issues/2960#issuecomment-1148768300

Guide contributeur

Direction de recherche
Étudiez l'implémentation de la fenêtre Comments dans Cutter (probablement dans src/gui/CommentsWidget.cpp ou similaire). Vérifiez comment les commentaires sont récupérés depuis Rizin et affichés via QString de Qt. Le problème suggère que les caractères Unicode non échappés ne sont pas affichés ; peut être que la chaîne est échappée avant l'affichage. Examinez la sortie 'CCl' de Rizin et comparez la aux données brutes des commentaires. Assurez vous que le texte est traité en UTF 8 et non en ASCII. Envisagez d'utiliser QTextCodec ou QString::fromUtf8. Reportez vous à l'issue liée #2960 pour le contexte.
Stack technique
cpp
Domaine
securitytooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70