diff options
| author |  Felipe Contreras <felipe.contreras@gmail.com> | 2013-12-07 07:09:41 -0600 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2013-12-09 13:18:43 -0800 |
| commit | 0c0ebc1fdf57c3ac968e1afdadbc17b9010da6fa (patch) | |
| tree | af6a810259c1290b8e6bbbcbe5e3daf8827d01fe /contrib/remote-helpers/git-remote-bzr | |
| parent | remote-hg: avoid buggy strftime() (diff) | |
| download | tgif-0c0ebc1fdf57c3ac968e1afdadbc17b9010da6fa.tar.xz | |
remote-helpers: add extra safety checks
Suggested-by: Roman Ovchinnikov <coolthecold@gmail.com>
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib/remote-helpers/git-remote-bzr')
| -rwxr-xr-x | contrib/remote-helpers/git-remote-bzr | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/contrib/remote-helpers/git-remote-bzr b/contrib/remote-helpers/git-remote-bzr index 7e345320ad..332aba784b 100755 --- a/contrib/remote-helpers/git-remote-bzr +++ b/contrib/remote-helpers/git-remote-bzr @@ -883,6 +883,16 @@ def main(args): global branches, peers global transports + marks = None + is_tmp = False + gitdir = os.environ.get('GIT_DIR', None) + + if len(args) < 3: + die('Not enough arguments.') + + if not gitdir: + die('GIT_DIR not set') + alias = args[1] url = args[2] @@ -891,7 +901,6 @@ def main(args): blob_marks = {} parsed_refs = {} files_cache = {} - marks = None branches = {} peers = {} transports = [] @@ -899,11 +908,8 @@ def main(args): if alias[5:] == url: is_tmp = True alias = hashlib.sha1(alias).hexdigest() - else: - is_tmp = False prefix = 'refs/bzr/%s' % alias - gitdir = os.environ['GIT_DIR'] dirname = os.path.join(gitdir, 'bzr', alias) if not is_tmp: |