[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [V5 PATCH 6/8] virtio-9p: Support for creating spec
From: |
M. Mohan Kumar |
Subject: |
Re: [Qemu-devel] Re: [V5 PATCH 6/8] virtio-9p: Support for creating special files |
Date: |
Fri, 18 Feb 2011 11:28:37 +0530 |
User-agent: |
KMail/1.13.5 (Linux/2.6.35.10-74.fc14.i686.PAE; KDE/4.5.4; i686; ; ) |
On Thursday 17 February 2011 4:19:17 pm Stefan Hajnoczi wrote:
> On Wed, Feb 16, 2011 at 12:23 PM, M. Mohan Kumar <address@hidden> wrote:
> > + switch (type) {
> > + case T_MKNOD:
> > + request.data.type = T_MKNOD;
> > + break;
> > + case T_MKDIR:
> > + request.data.type = T_MKDIR;
> > + break;
> > + case T_SYMLINK:
> > + request.data.type = T_SYMLINK;
> > + break;
> > + case T_LINK:
> > + request.data.type = T_LINK;
> > + break;
> > + }
>
> Or just request.data.type = type?
>
Oops, I will modify.
> > + if (oldpath) {
> > + request.data.oldpath_len = strlen(oldpath);
> > + strcpy(request.path.old_path, oldpath);
>
> It's not obvious that this strcpy() is safe.
>
I will change the code.
> > + }
> > + retval = v9fs_create_special(fs_ctx, &request, &error);
> > + if (retval < 0) {
> > + errno = error;
> > + return 0;
>
> This looks suspicious. Should the return value be negative on error?
Yes, it should return negative on error, I will update.
>
> > + char *tmp = qemu_strdup(rpath(fs_ctx, oldpath));
> > + if (tmp == NULL) {
>
> QEMU strdup never returns NULL.
Ok.
>
> Stefan
----
M. Mohan Kumar
- [Qemu-devel] [V5 PATCH 5/8] virtio-9p: Create support in chroot environment, (continued)
- [Qemu-devel] [V5 PATCH 5/8] virtio-9p: Create support in chroot environment, M. Mohan Kumar, 2011/02/16
- [Qemu-devel] [V5 PATCH 3/8] virtio-9p: Add client side interfaces for chroot environment, M. Mohan Kumar, 2011/02/16
- [Qemu-devel] [V5 PATCH 8/8] virtio-9p: Chroot environment for other functions, M. Mohan Kumar, 2011/02/16
- [Qemu-devel] [V5 PATCH 2/8] virtio-9p: Provide chroot environment server side interfaces, M. Mohan Kumar, 2011/02/16
- [Qemu-devel] [V5 PATCH 4/8] virtio-9p: Add support to open a file in chroot environment, M. Mohan Kumar, 2011/02/16
- [Qemu-devel] [V5 PATCH 6/8] virtio-9p: Support for creating special files, M. Mohan Kumar, 2011/02/16
- [Qemu-devel] [V5 PATCH 1/8] Implement qemu_read_full, M. Mohan Kumar, 2011/02/16