yosshin4004 / x68k_xsp Goto Github PK

GCCのGASに移植している間に、バグが発見されたかもしれません。　READMEによると、もし_xsp_divy_getの引数はだめだったら、返り値は-1です。　_xsp_divy_getのコードの中に、引数のチェックの結果はだめだったら、@fにブランチします。
そのブランチは@@：を指しますが、_xsp_divy_getの中にその@@:レーベルが入っていません。　-1の返り値の変わりに、_xsp_min_divh_setの一番目の`@@:｀に行ってしまうと思います。下にご覧ください。

XSPfnc.s:L505

	move.w	A7ID+arg1_w(sp),d0	* d0 = i
	bmi	@f			* i < 0 なら bra
	cmp #6,d0
	bgt	@f			* 6 < i なら bra

まだデバガーで調査をしていないんですけど、move.w #-1, d0の上にあの@@:を書くのは正しそうです。　このコードは、-1の引数を送ると、min_divhは変な数に入れるかもしれません。

XSPfnc.s:L535

@@:　＜ー（i < 0 || i > 6）なら、ここから_xsp_divy_getが続くと思います。
	cmp #MIN_DIVH_MAX,d0
	ble @f				* #MIN_DIVH_MAX >= h なら bra
	*-----[ MIN_DIVH_MAX < h ]
		moveq.l #MIN_DIVH_MAX,d0	* h = MIN_DIVH_MAX
@@:

	move.w	d0,min_divh		* min_divh 設定

	rts

失礼しました。