> +     if (!in_interrupt())
> +             get_task_comm(comm, current);
> +     else
> +             strcpy(comm, "Interrupt");
> +
> +     mccontrol.channel = pti_control_channel;
> +     pti_control_channel = (pti_control_channel + 1) & 0x7f;
> +
> +     snprintf(control_frame, CONTROL_FRAME_LEN, control_format,
> mc->master,
> +             mc->channel, comm);

Is comm guaranteed to be null terminated ?


>  
> +     pr_debug("%s(%d): Called.\n", __func__, __LINE__);
> +

Unrelated random debug change
        tty->count++;
> @@ -491,8 +554,7 @@ int pti_char_release(struct inode *inode, struct
> file *filp)
>   * @param data: trace data to be written.
>   * @param len:  # of byte to write.
>   * @param ppose: Not used in this function implementation.
> - * @return int : # of bytes written, or error.  -EMSGSIZE is
> - *               returned if length is beyond 8k.
> + * @return int : # of bytes written, or error.
>   */

And this change belongs in the copying patch not this one !

Imported the relevant bits into my tree

Alan
_______________________________________________
Meego-kernel mailing list
[email protected]
http://lists.meego.com/listinfo/meego-kernel

Reply via email to