summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhilip Kelley <phkelley@hotmail.com>2014-06-27 11:51:35 -0400
committerPhilip Kelley <phkelley@hotmail.com>2014-06-27 12:03:27 -0400
commitbc8a0886853c17d0899e9148d5c11c2c231c3ab4 (patch)
tree5f0f954c793fcb664d52b0c5cea490d50b6dd360
parent0145afe635b57bfbc3950030a572187b0644c0c5 (diff)
downloadlibgit2-bc8a0886853c17d0899e9148d5c11c2c231c3ab4.tar.gz
Fix assert when receiving uncommon sideband packet
-rw-r--r--src/indexer.c5
-rw-r--r--src/transports/smart_protocol.c4
2 files changed, 8 insertions, 1 deletions
diff --git a/src/indexer.c b/src/indexer.c
index eb8b23e92..8daff3dc4 100644
--- a/src/indexer.c
+++ b/src/indexer.c
@@ -435,6 +435,8 @@ static int write_at(git_indexer *idx, const void *data, git_off_t offset, size_t
git_map map;
int error;
+ assert(data && size);
+
/* the offset needs to be at the beginning of the a page boundary */
page_start = (offset / page_size) * page_size;
page_offset = offset - page_start;
@@ -453,6 +455,9 @@ static int append_to_pack(git_indexer *idx, const void *data, size_t size)
{
git_off_t current_size = idx->pack->mwf.size;
+ if (!size)
+ return 0;
+
/* add the extra space we need at the end */
if (p_ftruncate(idx->pack->mwf.fd, current_size + size) < 0) {
giterr_system_set(errno);
diff --git a/src/transports/smart_protocol.c b/src/transports/smart_protocol.c
index a52aacc60..82891165f 100644
--- a/src/transports/smart_protocol.c
+++ b/src/transports/smart_protocol.c
@@ -592,7 +592,9 @@ int git_smart__download_pack(
}
} else if (pkt->type == GIT_PKT_DATA) {
git_pkt_data *p = (git_pkt_data *) pkt;
- error = writepack->append(writepack, p->data, p->len, stats);
+
+ if (p->len)
+ error = writepack->append(writepack, p->data, p->len, stats);
} else if (pkt->type == GIT_PKT_FLUSH) {
/* A flush indicates the end of the packfile */
git__free(pkt);