genControl .lt crashes when controller literal is 0

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

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

評価

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

調査の方向性

src/Codec/CBOR/Cuddle/CBOR/Gen.hs を開き、genControl から始めて、PTUInt の CtlOp.Lt ケースに注目してください。controller-literal-0 入力を再現し、クラッシュするのではなく、空の範囲に関する明確なエラーが報告されることを確認するとともに、既存の生成動作が変わらないことを確認してください。

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

説明

bug

Description

genControl crashes when the .lt operator is used with a controller literal of 0, because choose receives an invalid range.

Location

src/Codec/CBOR/Cuddle/CBOR/Gen.hs, genControl function

Details

(CtlOp.Lt, CTree.Literal (Value (VUInt n) _)) -> case target of
  CTree.Postlude PTUInt -> S . TInteger <$> choose (0, fromIntegral n - 1)

When n == 0, choose (0, fromIntegral n - 1) becomes choose (0, -1), which is an invalid range.

Suggested fix

Handle n == 0 explicitly with a clear error message about the empty range (there are no unsigned integers less than 0).

主要言語
Haskell
スター
8
フォーク
5
平均マージ
3日 12時間
マージ済み PR(30日)
4

環境構築

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

はじめの一歩

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

IntersectMBO/cuddle のほかの issue

IntersectMBO/cuddle の issue をすべて見る

似ている issue

Haskell の issue をもっと見る

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

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