LM: config buffer version check differs from the score::flatbuffers version check

オープン
#945 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

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

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
リファクタリング
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
cpp
領域
backend, security

調査の方向性

Read flatbuffer_config_loader.cpp at the cited checks and docs/module/safety_mgt/dependency_aou_handling.rst for the dependency triage context. The first step is a maintainer decision on whether LM's schema version check is equivalent to the score::flatbuffers check. Once decided, update the triage verdict as specified and close the issue.

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

説明

inspection finding

AoU aou_req__flatbuffers__verify_version (FlatBuffers buffer version check before access) of a dependency of Lifecycle is open in the dependency AoU triage (#923, docs/module/safety_mgt/dependency_aou_handling.rst).

Component: Launch Manager

Evidence: LM does not use the baselibs version check (version_reader / buffer_version, file identifier); lm_flatcfg.fbs has no file_identifier. It checks its own int32 schema_version after structural verification (flatbuffer_config_loader.cpp:48-62, 98).

Decision needed: Maintainer question: is LM's own schema_version check (after Verifier) accepted as equivalent, or must LM adopt the score::flatbuffers version check (major/minor + 4-char identifier, present in baselibs v0.2.16) before accessing buffer data?

Once decided, the verdict in the triage page is updated (fulfils link, forwarded AoU, or a documented deviation) and this issue is closed.

主要言語
C++
スター
7
フォーク
37
平均マージ
2日 11時間
マージ済み PR(30日)
106

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

  • Dockerfile・Docker Compose ファイルなし
  • プルリクエストのテンプレートあり
  • コントリビューションガイドなし

はじめの一歩

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

eclipse-score/lifecycle のほかの issue

eclipse-score/lifecycle の issue をすべて見る

似ている issue

C++ の issue をもっと見る

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

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