Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Wrong return value in aarch64 disassembler operation_to_str() function

オープン 初心者向け
#8,589 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 2 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
78/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
c

調査の方向性

operation_to_str と aarch64_disassemble を見つけ、次に操作文字列がどのように選択され、NULL がどのように処理されるかを調べます。operation_to_str を直接呼び出している test.c を確認し、enum と文字列配列のチェックを検証します。他の to-string 関数と戻り値の動作が一貫し、既存の直接呼び出し元を壊していなければ完了です。

索引モデルが issue の本文から書いたものです。

説明

The aarch64_get_operation calls operation_to_str which doesn't return a NULL value, as is common for other to string functions in the source code.

But aarch64_disassemble then verifies for NULL value here, which is always false:

	const char *operation = aarch64_get_operation(instruction);
	if (operation == NULL)
		return FAILED_TO_DISASSEMBLE_OPERATION;

I think the fix should be operation_to_str to return a NULL to keep API consistent. test.c calls it directly without checking for the NULL but it shouldn't be submitting wrong values because previous checks, as long the enum and the string array are in sync.

主要言語
C++
スター
1.3k
フォーク
297
平均マージ
2日 13時間
マージ済み PR(30日)
16

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Vector35/binaryninja-api のほかの issue

Vector35/binaryninja-api の issue をすべて見る

似ている issue

C++ の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。