Skip to content

Commit

Permalink
frontend: comptime array slice-by-length OOB detection
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewrk committed Mar 21, 2024
1 parent 69dd1f9 commit 3990c8e
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 2 deletions.
15 changes: 15 additions & 0 deletions compile_errors/out of bounds array slice by length.zig
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
export fn b() void {
var buf: [5]u8 = undefined;
_ = buf[foo(6)..][0..10];
return error.TestFailed;
}

fn foo(a: u32) u32 {
return a;
}

// error
// backend=stage2
// target=native
//
// :3:26: error: slice end index 10 exceeds array length of type '[5]u8'
4 changes: 2 additions & 2 deletions safety/out of bounds array slice by length.zig
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,14 @@ const std = @import("std");

pub fn panic(message: []const u8, stack_trace: ?*std.builtin.StackTrace, _: ?usize) noreturn {
_ = stack_trace;
if (std.mem.eql(u8, message, "index out of bounds: index 16, len 5")) {
if (std.mem.eql(u8, message, "index out of bounds: index 9, len 5")) {
std.process.exit(0);
}
std.process.exit(1);
}
pub fn main() !void {
var buf: [5]u8 = undefined;
_ = buf[foo(6)..][0..10];
_ = buf[foo(6)..][0..3];
return error.TestFailed;
}
fn foo(a: u32) u32 {
Expand Down

0 comments on commit 3990c8e

Please sign in to comment.