AddressLookupTable: deserializeOwned

Author: Sobeston

src/replay/resolve_lookup.zig

@@ -300,10 +300,10 @@ fn getLookupTable(
     const account_bytes = buf[0..account.data.len()];
     account.data.readAll(account_bytes);
 
-    var table = AddressLookupTable.deserialize(account_bytes) catch {
+    const table = AddressLookupTable.deserializeOwned(allocator, account_bytes) catch |err| {
+        if (err == error.OutOfMemory) return err;
         return error.InvalidAddressLookupTableData;
     };
-    table.addresses = try allocator.dupe(Pubkey, table.addresses);
 
     // NOTE: deactivated lookup tables are allowed to be used,
     // according to agave's implementation. see here, where agave

src/runtime/program/address_lookup_table/state.zig

@@ -103,10 +103,10 @@ pub const AddressLookupTable = struct {
 
     // [agave] https://github.com/anza-xyz/agave/blob/d300f3733f45d64a3b6b9fdb5a1157f378e181c2/sdk/program/src/address_lookup_table/state.rs#L224
     /// NOTE: This AddressLookupTable's 'addresses' slice will point to inside 'data' - consider
-    /// if you need to clone the buffer.
+    /// if you need to clone the buffer (see deserializeOwned).
     pub fn deserialize(
         data: []const u8,
-    ) (error{OutOfMemory} || InstructionError)!AddressLookupTable {
+    ) error{ UninitializedAccount, InvalidAccountData }!AddressLookupTable {
         const noalloc = sig.utils.allocators.failing.allocator(.{});
         const state = sig.bincode.readFromSlice(noalloc, ProgramState, data, .{}) catch
             return error.InvalidAccountData;
@@ -124,4 +124,14 @@ pub const AddressLookupTable = struct {
             .addresses = addresses,
         };
     }
+
+    /// Deserializes an AddressLookupTable, coping .addresses into a new buffer.
+    pub fn deserializeOwned(
+        allocator: std.mem.Allocator,
+        data: []const u8,
+    ) error{ OutOfMemory, UninitializedAccount, InvalidAccountData }!AddressLookupTable {
+        var table = try AddressLookupTable.deserialize(data);
+        table.addresses = try allocator.dupe(Pubkey, table.addresses);
+        return table;
+    }
 };