Tom Zanussi <[EMAIL PROTECTED]> wrote: > > +static ssize_t relayfs_read(struct file *filp, > + char __user *buffer, > + size_t count, > + loff_t *ppos) > +{ > + struct inode *inode = filp->f_dentry->d_inode; > + struct rchan_buf *buf = RELAYFS_I(inode)->buf; > + unsigned int read_start, read_end, avail, start_subbuf; > + unsigned int buf_size = buf->chan->subbuf_size * buf->chan->n_subbufs; > + void *from; > + > + avail = relayfs_read_avail(buf, &start_subbuf); > + if (*ppos >= avail) > + return 0; > + > + read_start = relayfs_read_start(*ppos, avail, start_subbuf, buf); > + if (read_start == 0 && *ppos) > + return 0; > + > + read_end = relayfs_read_end(read_start, avail, start_subbuf, buf); > + if (read_end == read_start) > + return 0; > + > + from = buf->start + start_subbuf * buf->chan->subbuf_size + read_start; > + if (from >= buf->start + buf_size) > + from -= buf_size; > + > + count = min(count, read_end - read_start); > + if (copy_to_user(buffer, from, count)) > + return -EFAULT; > + > + *ppos = read_start + count; > + > + return count; > +}
The use of `unsigned int' in here will cause >4G reads to fail on 64-bit platforms. I think you want size_t throughout. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/