Skip to content

Commit 6902c4d

Browse files
committed
Merge branch 'rs/deflate-init-cleanup'
Code simplification. * rs/deflate-init-cleanup: zlib: initialize git_zstream in git_deflate_init{,_gzip,_raw}
2 parents 52cae64 + 9a6f128 commit 6902c4d

File tree

10 files changed

+2
-13
lines changed

10 files changed

+2
-13
lines changed

archive-zip.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,6 @@ static void *zlib_deflate_raw(void *data, unsigned long size,
120120
void *buffer;
121121
int result;
122122

123-
memset(&stream, 0, sizeof(stream));
124123
git_deflate_init_raw(&stream, compression_level);
125124
maxsize = git_deflate_bound(&stream, size);
126125
buffer = xmalloc(maxsize);
@@ -349,7 +348,6 @@ static int write_zip_entry(struct archiver_args *args,
349348
size_t out_len;
350349
unsigned char compressed[STREAM_BUFFER_SIZE * 2];
351350

352-
memset(&zstream, 0, sizeof(zstream));
353351
git_deflate_init_raw(&zstream, args->compression_level);
354352

355353
compressed_size = 0;

builtin/index-pack.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1204,7 +1204,6 @@ static int write_compressed(struct sha1file *f, void *in, unsigned int size)
12041204
int status;
12051205
unsigned char outbuf[4096];
12061206

1207-
memset(&stream, 0, sizeof(stream));
12081207
git_deflate_init(&stream, zlib_compression_level);
12091208
stream.next_in = in;
12101209
stream.avail_in = size;

builtin/pack-objects.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,6 @@ static unsigned long do_compress(void **pptr, unsigned long size)
125125
void *in, *out;
126126
unsigned long maxsize;
127127

128-
memset(&stream, 0, sizeof(stream));
129128
git_deflate_init(&stream, pack_compression_level);
130129
maxsize = git_deflate_bound(&stream, size);
131130

@@ -153,7 +152,6 @@ static unsigned long write_large_blob_data(struct git_istream *st, struct sha1fi
153152
unsigned char obuf[1024 * 16];
154153
unsigned long olen = 0;
155154

156-
memset(&stream, 0, sizeof(stream));
157155
git_deflate_init(&stream, pack_compression_level);
158156

159157
for (;;) {

bulk-checkin.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,6 @@ static int stream_to_pack(struct bulk_checkin_state *state,
105105
int write_object = (flags & HASH_WRITE_OBJECT);
106106
off_t offset = 0;
107107

108-
memset(&s, 0, sizeof(s));
109108
git_deflate_init(&s, pack_compression_level);
110109

111110
hdrlen = encode_in_pack_object_header(type, size, obuf);

diff.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2093,7 +2093,6 @@ static unsigned char *deflate_it(char *data,
20932093
unsigned char *deflated;
20942094
git_zstream stream;
20952095

2096-
memset(&stream, 0, sizeof(stream));
20972096
git_deflate_init(&stream, zlib_compression_level);
20982097
bound = git_deflate_bound(&stream, size);
20992098
deflated = xmalloc(bound);

fast-import.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1062,7 +1062,6 @@ static int store_object(
10621062
} else
10631063
delta = NULL;
10641064

1065-
memset(&s, 0, sizeof(s));
10661065
git_deflate_init(&s, pack_compression_level);
10671066
if (delta) {
10681067
s.next_in = delta;
@@ -1090,7 +1089,6 @@ static int store_object(
10901089
free(delta);
10911090
delta = NULL;
10921091

1093-
memset(&s, 0, sizeof(s));
10941092
git_deflate_init(&s, pack_compression_level);
10951093
s.next_in = (void *)dat->buf;
10961094
s.avail_in = dat->len;
@@ -1190,7 +1188,6 @@ static void stream_blob(uintmax_t len, unsigned char *sha1out, uintmax_t mark)
11901188

11911189
crc32_begin(pack_file);
11921190

1193-
memset(&s, 0, sizeof(s));
11941191
git_deflate_init(&s, pack_compression_level);
11951192

11961193
hdrlen = encode_in_pack_object_header(OBJ_BLOB, len, out_buf);

http-push.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,6 @@ static void start_put(struct transfer_request *request)
365365
hdrlen = sprintf(hdr, "%s %lu", typename(type), len) + 1;
366366

367367
/* Set it up */
368-
memset(&stream, 0, sizeof(stream));
369368
git_deflate_init(&stream, zlib_compression_level);
370369
size = git_deflate_bound(&stream, len + hdrlen);
371370
strbuf_init(&request->buffer.buf, size);

remote-curl.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,6 @@ static int post_rpc(struct rpc_state *rpc)
567567
git_zstream stream;
568568
int ret;
569569

570-
memset(&stream, 0, sizeof(stream));
571570
git_deflate_init_gzip(&stream, Z_BEST_COMPRESSION);
572571
gzip_size = git_deflate_bound(&stream, rpc->len);
573572
gzip_body = xmalloc(gzip_size);

sha1_file.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2943,7 +2943,6 @@ static int write_loose_object(const unsigned char *sha1, char *hdr, int hdrlen,
29432943
}
29442944

29452945
/* Set it up */
2946-
memset(&stream, 0, sizeof(stream));
29472946
git_deflate_init(&stream, zlib_compression_level);
29482947
stream.next_out = compressed;
29492948
stream.avail_out = sizeof(compressed);

zlib.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,7 @@ void git_deflate_init(git_zstream *strm, int level)
159159
{
160160
int status;
161161

162+
memset(strm, 0, sizeof(*strm));
162163
zlib_pre_call(strm);
163164
status = deflateInit(&strm->z, level);
164165
zlib_post_call(strm);
@@ -172,6 +173,7 @@ static void do_git_deflate_init(git_zstream *strm, int level, int windowBits)
172173
{
173174
int status;
174175

176+
memset(strm, 0, sizeof(*strm));
175177
zlib_pre_call(strm);
176178
status = deflateInit2(&strm->z, level,
177179
Z_DEFLATED, windowBits,

0 commit comments

Comments
 (0)