Summary

xsubstring with `to' argument less than string length results in a crash

Metadata

• Id: 1df5ee724bfc219b7e3d1f8f6f981d0f1a0335e0
• Trac id: 1590
• Type: defect
• Reporter: dieggsy
• Owner:
• Cc:
• Status: closed
• Component: extensions
• Estimated difficulty: easy
• Resolution: fixed
• Priority: major
• Milestone: someday
• Version: 5.0.0
• Changetime: 2021-03-13 16:08:29 UTC
• Created: 2019-03-21 03:28:25 UTC
• Keywords: srfi13,xsubstring

Attachments

• 1df5ee724bfc219b7e3d1f8f6f981d0f1a0335e0/attachments/srfi-13.patch

Description

I would expect:

 (xsubstring "foo" 0 2) => "fo"

But I get:

 #;4> (xsubstring "foo" 0 2)
 [panic] Detected corrupted data in stack - execution terminated
 
 <syntax>
 <eval>  <--

Changes and comments

[2019-03-21 03:30:40 UTC] dieggsy changed component from core libraries to extensions

[2019-08-27 16:40:02 UTC] dieggsy set milestone to someday

[2021-02-27 03:12:57 UTC] dieggsy attached srfi-13.patch (description=#f)

[2021-02-27 03:13:38 UTC] dieggsy wrote:

I believe the attached patch fixes the issue.

[2021-03-04 04:40:20 UTC] dieggsy set difficulty to easy

[2021-03-04 04:40:20 UTC] dieggsy changed milestone from someday to 5.3

[2021-03-06 16:26:06 UTC] dieggsy changed milestone from 5.3 to someday

[2021-03-13 16:08:29 UTC] kooda changed status from new to closed

[2021-03-13 16:08:29 UTC] kooda set resolution to fixed

[2021-03-13 16:08:29 UTC] kooda wrote:

Fixed in 0.3.1, thanks for the patch!