authorgravatar for lgustaf1@binghamton.eduLayne Gustafson <lgustaf1@binghamton.edu> 2020-01-08 23:14:25-05:00
committergravatar for andrew@ziglang.orgAndrew Kelley <andrew@ziglang.org> 2020-01-19 20:53:20-05:00
logebb6f15bbad7d86c76bd1e8deb39553b5791a69e
treebcf1f354b293d0d51f2e74ea144ce2a488ab1dea
parentfd17a9962b07147a5b20487ab8e4d9bc0aa946cd
signature Commit is signed but in an unrecognized format.

Make sure llvm strings are null-terminated

1 files changed, 16 insertions(+), 4 deletions(-)

src-self-hosted/stage1.zig+16-4
......@@ -617,6 +617,15 @@ fn printCpusForArch(arch_name: []const u8, show_dependencies: bool) !void {
617617 }
618618}
619619
620fn toNullTerminatedStringAlloc(allocator: *std.mem.Allocator, str: []const u8) ![:0]const u8 {
621 var buffer = try std.Buffer.init(allocator, str);
622
623 const len = buffer.len();
624
625 // Don't deinit since we steal all the buffer's memory here.
626 return buffer.list.toOwnedSlice()[0..len :0];
627}
628
620629const Stage2TargetDetails = struct {
621630 allocator: *std.mem.Allocator,
622631 target_details: std.target.TargetDetails,
......@@ -643,8 +652,8 @@ const Stage2TargetDetails = struct {
643652 .target_details = .{
644653 .cpu = cpu,
645654 },
646 .llvm_cpu_str = cpu.name,
647 .llvm_features_str = "",
655 .llvm_cpu_str = try toNullTerminatedStringAlloc(allocator, cpu.name),
656 .llvm_features_str = try toNullTerminatedStringAlloc(allocator, ""),
648657 .builtin_str = builtin_str_buffer.toOwnedSlice(),
649658 };
650659 }
......@@ -672,13 +681,16 @@ const Stage2TargetDetails = struct {
672681
673682 try builtin_str_buffer.append("}};");
674683
684 // This is needed here because llvm_features_buffer.len() is no longer valid after toOwnedSlice().
685 const llvm_features_buffer_len = llvm_features_buffer.len();
686
675687 return Self{
676688 .allocator = allocator,
677689 .target_details = std.target.TargetDetails{
678690 .features = features,
679691 },
680 .llvm_cpu_str = "",
681 .llvm_features_str = llvm_features_buffer.toOwnedSlice(),
692 .llvm_cpu_str = try toNullTerminatedStringAlloc(allocator, ""),
693 .llvm_features_str = llvm_features_buffer.toOwnedSlice()[0..llvm_features_buffer_len :0],
682694 .builtin_str = builtin_str_buffer.toOwnedSlice(),
683695 };
684696 }