Dave Hansen wrote:
> Pasting from the applied patches:
>> /* cr_write_file - dump the state of a given file pointer */
>> int cr_write_file(struct cr_ctx *ctx, struct file *file)
>> {
>> struct cr_hdr h;
>> struct cr_hdr_file *hh;
>> struct inode *inode = file->f_dentry->d_inode;
>> enum fd_type fd_type;
>> int new = 0; /* pacitfy gcc */
>> int ret;
>>
>> h.type = CR_HDR_FILE;
>> h.len = sizeof(*hh);
>>
>> hh = cr_hbuf_get(ctx, sizeof(*hh));
>> if (!hh)
>> return -ENOMEM;
>>
>> hh->f_flags = file->f_flags;
>> hh->f_mode = file->f_mode;
>> hh->f_pos = file->f_pos;
>> hh->f_version = file->f_version;
>> /* FIX: need also file->uid, file->gid, file->f_owner, etc */
>>
>> fd_type = cr_inode_to_fdtype(inode);
>> if (fd_type < 0) {
>> cr_hbuf_put(ctx, sizeof(*hh));
>> return fd_type;
>> }
>>
>> hh->fd_objref = cr_inode_to_objref(ctx, inode, hh->fd_type, &new);
>> pr_debug("type %d objref %d (%d)\n", hh->fd_type, hh->fd_objref, new);
>> if (!new)
>> fd_type = CR_FD_OBJREF;
>
> Where does 'hh->fd_type' get initialized? Did you mean just plain
> 'fd_type'?
Try one more line ...
Oren.
_______________________________________________
Containers mailing list
Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/containers
