[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/28] 9p CVE-2016-9602 fixes 2017-02-28 for 2.9
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PULL 00/28] 9p CVE-2016-9602 fixes 2017-02-28 for 2.9 soft freeze |
Date: |
Tue, 28 Feb 2017 16:11:04 +0100 |
On Tue, 28 Feb 2017 17:55:02 +0300
Michael Tokarev <address@hidden> wrote:
> 28.02.2017 17:22, Greg Kurz wrote:
> > On Tue, 28 Feb 2017 17:02:44 +0300
> > Michael Tokarev <address@hidden> wrote:
> >
> >> 28.02.2017 13:30, Greg Kurz wrote:
> >>> The following changes since commit
> >>> 9b9fbe8a4e9eec9072ee2697a6af59144442785f:
> >>>
> >>> Merge remote-tracking branch 'remotes/kraxel/tags/pull-ui-20170227-1'
> >>> into staging (2017-02-27 19:19:46 +0000)
> >>>
> >>> are available in the git repository at:
> >>>
> >>> https://github.com/gkurz/qemu.git tags/cve-2016-9602-for-upstream
> >>>
> >>> for you to fetch changes up to c23d5f1d5bc0e23aeb845b1af8f996f16783ce98:
> >>>
> >>
> >> Greg, did you forget to push maybe? There's no tag
> >> "cve-2016-9602-for-upstream"
> >> and no object c23d5f1d5bc0e23aeb845b1af8f996f16783ce98.
> >>
> >
> > I had pushed actually and...
> >
> > https://github.com/gkurz/qemu/commits/cve-2016-9602-for-upstream
> > https://github.com/gkurz/qemu/commit/c23d5f1d5bc0e23aeb845b1af8f996f16783ce98
> >
>
> Interesting. Perhaps I've never worked with github before.
> It works when referring to particular commit like this.
> but the tag isn't visible in github UI, and neither the
> tag nor this commit ID is visible when cloning github
> repository locally. I wonder where's that.
>
Yeah I confirm that's the way it goes with github and I was pretty
surprised myself when I first realized that... but I must confess
I never tried to investigate.
> $ git remote -v | grep gkurz
> gkurz git://github.com/gkurz/qemu.git (fetch)
> gkurz git://github.com/gkurz/qemu.git (push)
> $ git remote update gkurz
> Fetching gkurz
> $ git show c23d5f1d5bc0e23aeb845b1af8f996f16783ce98
> fatal: bad object c23d5f1d5bc0e23aeb845b1af8f996f16783ce98
> $ git show cve-2016-9602-for-upstream --
> fatal: bad revision 'cve-2016-9602-for-upstream'
>
> that's now. I'll dig into that later, there's apparently
> nothing wrong on your side, I'm sorry for the noise.
>
No problem. I've just verified I could merge these two pull
requests in a clean 'git clone' of master, and it works as
expected... phew! :)
Cheers.
--
Greg
> Thanks,
>
> /mjt
pgp5OnqNwTvsc.pgp
Description: OpenPGP digital signature
- [Qemu-devel] [PULL 22/28] 9pfs: local: chmod: don't follow symlinks, (continued)
- [Qemu-devel] [PULL 22/28] 9pfs: local: chmod: don't follow symlinks, Greg Kurz, 2017/02/28
- [Qemu-devel] [PULL 23/28] 9pfs: local: chown: don't follow symlinks, Greg Kurz, 2017/02/28
- [Qemu-devel] [PULL 24/28] 9pfs: local: symlink: don't follow symlinks, Greg Kurz, 2017/02/28
- [Qemu-devel] [PULL 21/28] 9pfs: local: link: don't follow symlinks, Greg Kurz, 2017/02/28
- [Qemu-devel] [PULL 28/28] 9pfs: local: drop unused code, Greg Kurz, 2017/02/28
- [Qemu-devel] [PULL 26/28] 9pfs: local: mkdir: don't follow symlinks, Greg Kurz, 2017/02/28
- [Qemu-devel] [PULL 27/28] 9pfs: local: open2: don't follow symlinks, Greg Kurz, 2017/02/28
- Re: [Qemu-devel] [PULL 00/28] 9p CVE-2016-9602 fixes 2017-02-28 for 2.9 soft freeze, Michael Tokarev, 2017/02/28