Fix double free in error case

Related to #153.

Author: jstrieb

src/statistics.zig

@@ -240,7 +240,7 @@ fn getReposByYear(
         "Getting {d} month{s} of data starting from {d}/{d}...",
         .{ months, if (months != 1) "s" else "", start_month + 1, year },
     );
-    var response = try context.client.graphql(
+    const response = try context.client.graphql(
         \\query ($from: DateTime, $to: DateTime) {
         \\  viewer {
         \\    contributionsCollection(from: $from, to: $to) {
@@ -289,7 +289,7 @@ fn getReposByYear(
             ),
         },
     );
-    errdefer context.client.allocator.free(response.body);
+    defer context.client.allocator.free(response.body);
     if (response.status != .ok) {
         std.log.err(
             "Failed to get data from {d} ({?s})",
@@ -328,7 +328,6 @@ fn getReposByYear(
         response.body,
         .{ .ignore_unknown_fields = true, .allocate = .alloc_always },
     )).data.viewer.contributionsCollection;
-    context.client.allocator.free(response.body);
     std.log.info(
         "Parsed {d} total repositories from {d}",
         .{ stats.commitContributionsByRepository.len, year },
@@ -426,7 +425,7 @@ fn getReposByYear(
             "Getting views for {s}...",
             .{raw_repo.nameWithOwner},
         );
-        response = try context.client.rest(
+        const response2 = try context.client.rest(
             try std.mem.concat(
                 context.arena.allocator(),
                 u8,
@@ -437,18 +436,18 @@ fn getReposByYear(
                 },
             ),
         );
-        defer context.client.allocator.free(response.body);
-        if (response.status == .ok) {
+        defer context.client.allocator.free(response2.body);
+        if (response2.status == .ok) {
             repository.views = (try std.json.parseFromSliceLeaky(
                 struct { count: u32 },
                 context.arena.allocator(),
-                response.body,
+                response2.body,
                 .{ .ignore_unknown_fields = true },
             )).count;
         } else {
             std.log.info(
                 "Failed to get views for {s} ({?s})",
-                .{ raw_repo.nameWithOwner, response.status.phrase() },
+                .{ raw_repo.nameWithOwner, response2.status.phrase() },
             );
         }