LDD3 short.c 问题

wzw200

irqreturn_t short_interrupt(int irq, void *dev_id, struct pt_regs *regs)
{
        struct timeval tv;
        int written;

        do_gettimeofday(&tv);

        written = sprintf((char *)short_head,"%08u.%06u\n",(int)(tv.tv_sec % 100000000), (int)(tv.tv_usec));  //这个不是有可能越界吗，为什么还这样用啊，
       
        short_incr_bp(&short_head, written);
        wake_up_interruptible(&short_queue); /* awake any reading process */
        return IRQ_HANDLED;
}

整个程序我没有运行起来，只是看看，

自己实验的：

unsigned long short_buffer = 0;
int short_init(void)
{
    int i=0;
    short_buffer = __get_free_pages(GFP_KERNEL,0);
    i=sprintf((char*)short_buffer+1024*1024*1024,"%d",1);  //出问题了
    printk("init----------end:%d\n",i);
    return 0;
}

wzw200

为什么没有人呢，我反复看了这个例子，他不断的 short_incr_bp(&short_head, written);，觉得早晚会越界啊，不知道理解对不对，

Tinnal

回复 2# wzw200


你的程序都不全，要人怎么看！还要别人先给构把LDD的原码搜出来吗。要问问题，先把涉及的关键代码都贴出来。

   

mnipxh

short_incr_bp中处理了越界的情况，注意是16字节对齐的。

lz看代码比我还不仔细。

wzw200

回复 4# mnipxh

你看清楚了，上面是先sprintf 之后才处理的，如果我这一次SPRINTF越界了，再处理吗，他是一个环

wzw200

上传了代码，把 short.gz 修改成 short.c就可以了，代码只用到了629行的中断，其它的中断相关的可以不用看

wzw200

回复 3# Tinnal

上传了代码有空看看吧，谢谢
   

黎明748

怎么会越界。。。

黎明748

"%08u.%06u\n"是16个字节.4096/16=256.所以sprintf是不会越界的.new最大值为short_buffer+PAGE_SIZE。不会大于short_buffer+PAGE_SIZE。除非出错

wzw200

谢谢楼上几回的回复，我想明白了，16对齐，是这样的，