Commit 65e4ab7a by Edward Thomson Committed by GitHub

Merge pull request #3814 from pks-t/pks/invalid-memrefs

Fix invalid memory references
parents f0ee795c 13deb874
...@@ -505,10 +505,11 @@ static int index_remove_entry(git_index *index, size_t pos) ...@@ -505,10 +505,11 @@ static int index_remove_entry(git_index *index, size_t pos)
int error = 0; int error = 0;
git_index_entry *entry = git_vector_get(&index->entries, pos); git_index_entry *entry = git_vector_get(&index->entries, pos);
if (entry != NULL) if (entry != NULL) {
git_tree_cache_invalidate_path(index->tree, entry->path); git_tree_cache_invalidate_path(index->tree, entry->path);
DELETE_IN_MAP(index, entry);
}
DELETE_IN_MAP(index, entry);
error = git_vector_remove(&index->entries, pos); error = git_vector_remove(&index->entries, pos);
if (!error) { if (!error) {
......
...@@ -433,6 +433,7 @@ int git_pkt_parse_line( ...@@ -433,6 +433,7 @@ int git_pkt_parse_line(
* line? * line?
*/ */
if (len == PKT_LEN_SIZE) { if (len == PKT_LEN_SIZE) {
*head = NULL;
*out = line; *out = line;
return 0; return 0;
} }
......
...@@ -759,6 +759,14 @@ static int add_push_report_sideband_pkt(git_push *push, git_pkt_data *data_pkt, ...@@ -759,6 +759,14 @@ static int add_push_report_sideband_pkt(git_push *push, git_pkt_data *data_pkt,
line_len -= (line_end - line); line_len -= (line_end - line);
line = line_end; line = line_end;
/* When a valid packet with no content has been
* read, git_pkt_parse_line does not report an
* error, but the pkt pointer has not been set.
* Handle this by skipping over empty packets.
*/
if (pkt == NULL)
continue;
error = add_push_report_pkt(push, pkt); error = add_push_report_pkt(push, pkt);
git_pkt_free(pkt); git_pkt_free(pkt);
...@@ -813,6 +821,9 @@ static int parse_report(transport_smart *transport, git_push *push) ...@@ -813,6 +821,9 @@ static int parse_report(transport_smart *transport, git_push *push)
error = 0; error = 0;
if (pkt == NULL)
continue;
switch (pkt->type) { switch (pkt->type) {
case GIT_PKT_DATA: case GIT_PKT_DATA:
/* This is a sideband packet which contains other packets */ /* This is a sideband packet which contains other packets */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment