|
3865f7f6
|
2012-12-27T23:23:12
|
|
Invalid ref name normalization leaked memory
When normalizing a reference name, if there is an error because
the name is invalid, then the memory allocated for storing the
name could be leaked if the caller was not careful and assumed
that the error return code meant that no allocation had occurred.
This fixes that by explicitly deallocating the reference name
buffer if there is an error in normalizing the name.
|
|
|
f616a36b
|
2012-12-27T22:25:52
|
|
Make spoolandsort a pushable iterator behavior
An earlier change to `git_diff_from_iterators` introduced a
memory leak where the allocated spoolandsort iterator was not
returned to the caller and thus not freed.
One proposal changes all iterator APIs to use git_iterator** so
we can reallocate the iterator at will, but that seems unexpected.
This commit makes it so that an iterator can be changed in place.
The callbacks are isolated in a separate structure and a pointer
to that structure can be reassigned by the spoolandsort extension.
This means that spoolandsort doesn't create a new iterator; it
just allocates a new block of callbacks (along with space for its
own extra data) and swaps that into the iterator.
Additionally, since spoolandsort is only needed to switch the
case sensitivity of an iterator, this simplifies the API to only
take the ignore_case boolean and to be a no-op if the iterator
already matches the requested case sensitivity.
|
|
83261a44
|
2012-12-26T16:08:10
|
|
Merge pull request #1173 from nulltoken/bug/discover
Teach UNC paths to git_path_dirname_r()
|
|
50a762a5
|
2012-12-26T12:03:07
|
|
path: Teach UNC paths to git_path_dirname_r()
Fix libgit2/libgit2sharp#256
|
|
|
34b6f05f
|
2012-12-26T11:59:07
|
|
path: enhance git_path_dirname_r() test coverage
|
|
7382551f
|
2012-12-22T16:29:59
|
|
Fix -Wmaybe-uninitialized warning
|
|
|
2052e3c0
|
2012-12-21T10:32:23
|
|
Merge pull request #1163 from barrbrain/check-exists-before-write
odb: check if object exists before writing
|
|
4d185dd9
|
2012-12-19T14:30:06
|
|
odb: check if object exists before writing
Update the procondition of git_odb_backend::write.
It may now be assumed that the object has already been hashed.
|
|
|
4c10d1af
|
2012-12-20T08:49:32
|
|
Merge pull request #1166 from scunz/antibiotica_for_travis
Mark travis build as failed if any test fails
|
|
cedc15af
|
2012-12-20T03:51:06
|
|
Mark travis build as failed if any test fails
|
|
92c4d258
|
2012-12-19T15:12:49
|
|
Merge pull request #1138 from ethomson/fetchhead
fetchhead reading/iterating
|
|
7fcec834
|
2012-12-11T22:31:21
|
|
fetchhead reading/iterating
|
|
|
5c3c86b0
|
2012-12-19T13:43:50
|
|
Merge pull request #1150 from schu/fix-netops-ssl
netops: on SSL teardown only send shutdown alert
|
|
|
0f7f1b1b
|
2012-12-19T10:26:54
|
|
Merge pull request #1159 from rick/be-consistent-be-be-consistent
Make `goto cleanup` more consistent
|
|
|
f7a40a82
|
2012-12-19T10:26:01
|
|
Merge pull request #1162 from scunz/cmake_cleanups
RFC: Some cleanups/features in CMakeLists.txt
|
|
|
6b459a1c
|
2012-12-19T10:25:04
|
|
Merge pull request #1164 from scunz/fixtestleak
Fix some leaks in tests
|
|
08a325a3
|
2012-12-19T12:52:14
|
|
reflog: Actual error handling
|
|
|
8a810441
|
2012-12-19T12:48:12
|
|
reflog: Rename error handling
|
|
|
d5cf4665
|
2012-12-19T08:04:31
|
|
Fix some leaks and (possibly) dangling pointers in tests
Also adds some asserts.
|
|
|
96a289f5
|
2012-12-19T05:24:23
|
|
clone-empty-test: Don't use one pointer for two things
... so we can clean up correctly.
|
|
|
3dc0207b
|
2012-12-19T05:21:11
|
|
revwalk-test: Don't leak the second repository
|
|
|
523a3ae5
|
2012-12-18T20:40:57
|
|
MSVC: Don't list all source files in an endless list
Instead tell MSVC to group the source files by directory.
|
|
|
94243295
|
2012-12-18T19:51:31
|
|
DRY: Add function that adds os-specific libraries to our targets
|
|
|
19a766a2
|
2012-12-18T19:32:31
|
|
Collect configuration options at the top of the file
- Also document the -DSTDCALL even better.
|
|
|
521479b1
|
2012-12-18T19:18:13
|
|
DRY: Don't repeat globs for libgit2's own source files
|
|
|
c5309eb2
|
2012-12-18T19:07:59
|
|
Remove special case source globs for Solaris
With the src/compat/*.c glob removed, there is no longer a difference to
the default globs we use for the IF( UNIX ) case.
|
|
|
49b63008
|
2012-12-18T19:07:08
|
|
Remove src/compat/*.c from source globs
This directory doesn't exist.
|
|
|
b53671ae
|
2012-12-18T19:03:29
|
|
Search for zlib unconditional
Up to now, on windows we don't even bother to look if the user has a zlib
available somwhere.
In almost all larger commercial projects i've participated in, it was not
at all uncommon to have such a dependency somewhere in the source tree and
use it whereever required.
Searching for it, even if it's unlikely to be present, allows for such a
scenario (i.e. by prefilling the CMake-Cache).
|
|
|
3d007f4f
|
2012-12-18T18:55:23
|
|
DRY: Scan for regex.c only in one place
|
|
|
8d457891
|
2012-12-18T18:52:49
|
|
Only add deps/http-parser to include-dirs if required
|
|
9ec50c25
|
2012-12-18T18:15:21
|
|
Make goto cleanup more consistent
There may be some question about whether this is likely to be needed
at all, but that's above my head at the moment.
|
|
|
9d8fa537
|
2012-12-18T15:48:17
|
|
Merge pull request #1158 from rick/i-do-not-think-this-means-what-you-think-it-means
don't deref before we've asserted
|
|
|
3a6420f3
|
2012-12-18T17:46:18
|
|
don't deref before we've asserted
just sayin'.
|
|
|
25fa3ae3
|
2012-12-18T14:18:04
|
|
Merge pull request #1157 from rick/trivial-comment-fix
Trivial comment fix
|
|
|
ed4e887d
|
2012-12-18T16:09:57
|
|
Also, whitespace.
I was totally flaunting @ben's 3-space tab advice.
|
|
|
33f169e2
|
2012-12-18T16:07:18
|
|
Improve comment text
This looked wrong to me. I *think* this is more appropriate
commentary.
|
|
|
957a3e36
|
2012-12-18T12:07:13
|
|
Merge pull request #1156 from scunz/another_testcrash
Reset global variable to NULL after free'ing resource
|
|
|
26290cd1
|
2012-12-18T19:52:37
|
|
Reset global variable to NULL after free'ing resource
|
|
|
e62171e2
|
2012-12-17T11:10:25
|
|
Merge pull request #1151 from arrbee/fix-diff-constructor-names
Fix diff constructor names
|
|
|
ba084f7a
|
2012-12-17T11:03:42
|
|
More diff.h comment fixes
Based on feedback from the ObjectiveGit folks, these are some
further updates to diff.h areas that are poorly documented.
|
|
|
56c72b75
|
2012-12-17T11:00:53
|
|
Fix diff constructor name order confusion
The diff constructor functions had some confusing names, where the
"old" side of the diff was coming after the "new" side. This
reverses the order in the function name to make it less confusing.
Specifically...
* git_diff_index_to_tree becomes git_diff_tree_to_index
* git_diff_workdir_to_index becomes git_diff_index_to_workdir
* git_diff_workdir_to_tree becomes git_diff_tree_to_workdir
|
|
|
f2b00cbd
|
2012-12-17T19:35:40
|
|
netops: on SSL teardown only send shutdown alert
According to man 3 SSL_shutdown / TLS, "If a unidirectional shutdown is
enough (the underlying connection shall be closed anyway), this first
call to SSL_shutdown() is sufficient."
Currently, an unidirectional shutdown is enough, since
gitno_ssl_teardown is called by gitno_close only. Do so to avoid further
errors (by misbehaving peers for example).
Fixes #1129.
|
|
|
9c8dbc88
|
2012-12-17T19:18:34
|
|
netops: properly handle GITNO_CONNECT_SSL_NO_CHECK_CERT
Don't return an error just because GITNO_CONNECT_SSL_NO_CHECK_CERT is
set.
|
|
|
0d10e79d
|
2012-12-17T10:13:36
|
|
Merge pull request #1149 from nulltoken/topic/blob_isbinary
Introduce git_blob_is_binary()
|
|
|
f7953509
|
2012-12-17T08:31:04
|
|
Merge pull request #1148 from scunz/test_fix
Cleanup the empty-clone test correctly
|
|
|
a3337f10
|
2012-12-17T15:15:20
|
|
blob: introduce git_blob_is_binary()
|
|
|
bdb94c21
|
2012-12-17T12:20:52
|
|
Fix MSVC compilation warnings
|
|
|
22d23c61
|
2012-12-17T17:15:58
|
|
Cleanup the test correctly
|
|
|
101659be
|
2012-12-17T15:50:12
|
|
Fix -Wmaybe-uninitialized warning
|
|
|
69a402d4
|
2012-12-17T03:17:16
|
|
Merge pull request #1141 from ben/clone-empty-repo
Allow clone to handle empty repos
|
|
c4e3e797
|
2012-12-16T12:27:11
|
|
Ensure static variables are nulled after every test
|
|
|
cc3e9b5a
|
2012-12-16T10:50:10
|
|
Make building samples more friendly
|
|
|
a7f125cd
|
2012-12-15T14:56:20
|
|
Fix fetchhead tests
|
|
|
2a2d1ab0
|
2012-12-15T14:30:20
|
|
Cloning empty repos: only allow missing target for HEAD
|
|
|
71131b55
|
2012-12-15T11:31:25
|
|
Merge pull request #1145 from esc/feature/mailmap
adding .mailmap file
|
|
d777a407
|
2012-12-15T18:20:22
|
|
adding .mailmap file
Help clarify who are the top commiters when using 'shortlog -sn'.
|
|
|
28abb187
|
2012-12-14T14:16:10
|
|
Stop returning incorrect error message
|
|
|
57f5d8dc
|
2012-12-14T14:15:42
|
|
Remove placeholder files during tests
|
|
|
1164acde
|
2012-12-14T14:00:35
|
|
Rebase fixup
|
|
|
b524fe1a
|
2012-12-14T08:35:59
|
|
Local Only ignore ENOTFOUNDs when adding corrupted refs
|
|
|
850b1edf
|
2012-12-13T12:55:28
|
|
Allow clone to handle empty repos
|
|
|
be5869fc
|
2012-12-14T13:53:50
|
|
Merge pull request #1143 from ben/clone-options
Options structure for git_clone
|
|
|
b9e7e2b4
|
2012-12-14T13:46:45
|
|
Move non-options back out of options struct
|
|
|
0015b587
|
2012-12-14T13:18:06
|
|
Deploy git_clone_options to network sample
|
|
|
18b2d560
|
2012-12-14T13:03:59
|
|
Deploy git_clone_options; remove git_clone_bare
|
|
|
7e610440
|
2012-12-14T12:21:59
|
|
Introduce git_clone_options
|
|
37ac4436
|
2012-12-14T03:02:33
|
|
travis: update the apt-get db
|
|
|
2d466b7d
|
2012-12-14T02:49:11
|
|
tests: Fix unused temp repo
|
|
|
509216a5
|
2012-12-14T02:49:01
|
|
test: Fix clone tests
|
|
|
b0b9fd32
|
2012-12-14T02:41:53
|
|
Merge remote-tracking branch 'origin/clone-auth' into development
|
|
|
44f5f777
|
2012-12-14T00:16:03
|
|
Merge remote-tracking branch 'jamill/transport_localpaths' into development
|
|
f0a2def5
|
2012-12-13T18:08:45
|
|
Fix comment so it doesn't go over 100 chars
|
|
|
8a8820d8
|
2012-12-13T17:58:12
|
|
Add test to clone with absolute path
|
|
|
2b10a2b0
|
2012-12-13T11:47:14
|
|
Enable authenticated clones in network sample
|
|
|
cb2ace69
|
2012-12-13T12:51:23
|
|
Transport resolution on Win32 should handle absolute local paths
|
|
|
24393ea6
|
2012-12-13T09:14:56
|
|
Stop premature remote freeing when cloning
|
|
|
34c8c754
|
2012-12-13T08:54:23
|
|
Fix network example
|
|
|
7c353afd
|
2012-12-13T08:47:29
|
|
Define constant for default fetch spec
|
|
|
ae0ddc6f
|
2012-12-12T19:53:05
|
|
Merge pull request #1139 from ethomson/index_bugfix
don't walk off the end of the index
|
|
|
44f36f6e
|
2012-12-12T19:48:44
|
|
Convert clone to use dangling remotes
|
|
|
e759b072
|
2012-12-12T17:54:12
|
|
don't walk off the end of the index
|
|
|
b914e17d
|
2012-12-12T12:23:24
|
|
API to set a dangling remote's repository
|
|
|
a71c27cc
|
2012-12-12T12:15:25
|
|
Allow creation of dangling remotes
|
|
|
6cacd44b
|
2012-12-12T06:47:50
|
|
Merge pull request #1136 from pclouds/tree-cache-counts
tree cache: loosen negative entry count check
|
|
08f3d6ca
|
2012-12-12T19:23:05
|
|
tree cache: loosen negative entry count check
While C Git has been writing entry count -1 (ie. never other negative
numbers) as invalid since day 1, it accepts all negative entry counts
as invalid. JGit follows the same rule. libgit2 should also follow, or
the index that works with C Git or JGit may someday be rejected by
libgit2.
Other reimplementations like dulwich and grit have not bothered with
parsing or writing tree cache.
|
|
|
8d13035e
|
2012-12-11T15:10:25
|
|
Merge pull request #1135 from ymendel/patch-1
ignore mkmf.log
|
|
64071617
|
2012-12-11T15:08:08
|
|
ignore mkmf.log
no build artifacts
no parents
|
|
|
55325efa
|
2012-12-10T15:54:15
|
|
Merge pull request #1133 from arrbee/more-iterator-cleanup
More iterator cleanup
|
|
|
91e7d263
|
2012-12-10T15:29:44
|
|
Fix iterator reset and add reset ranges
The `git_iterator_reset` command has not been working in all cases
particularly when there is a start and end range. This fixes it
and adds tests for it, and also extends it with the ability to
update the start/end range strings when an iterator is reset.
|
|
|
9950d27a
|
2012-12-06T13:26:58
|
|
Clean up iterator APIs
This removes the need to explicitly pass the repo into iterators
where the repo is implied by the other parameters. This moves
the repo to be owned by the parent struct. Also, this has some
iterator related updates to the internal diff API to lay the
groundwork for checkout improvements.
|
|
|
4cbe9a1b
|
2012-12-10T11:48:20
|
|
Add git_cred_acquire_cb payload to winhttp transport
|
|
|
a72a8f34
|
2012-12-10T11:40:49
|
|
Merge pull request #1132 from ben/cred-acquire-payload
Add a payload param to git_cred_acquire_cb
|
|
|
59bccf33
|
2012-12-10T11:11:01
|
|
Add a payload param to git_cred_acquire_cb
Fixes #1128.
|
|
|
72629a10
|
2012-12-10T10:05:31
|
|
Clean up GCC build warnings
|
|
|
22bcf86c
|
2012-12-10T06:55:59
|
|
Merge pull request #1126 from carlosmn/indexer-buffer
indexer: move the temporary buffers into the indexer object
|
|
|
cbad0270
|
2012-12-10T06:55:13
|
|
Merge pull request #1127 from libgit2/fix-mwindow-threading
Fix mwindow mutex initialization and error checking
|
|
1d009603
|
2012-12-09T02:40:16
|
|
orite C89
|
|
|
2bb1c7aa
|
2012-12-09T02:37:33
|
|
Treat git_mutex_lock as successful when threads are disabled
|
|
|
a35b3864
|
2012-12-09T02:31:39
|
|
Always check the result of git_mutex_lock
|
|
|
c3320aca
|
2012-12-09T02:22:50
|
|
git__mwindow_mutex needs to be initialized even with pthreads
This could also use PTHREAD_MUTEX_INITIALIZER, but a dynamic initializer seems like a more portable concept, and we won't need another #define on top of git_mutex_init()
|