handling unknown messages (pt 2 of TODO)

Iskren Chernev iskren.chernev at gmail.com
Wed Apr 13 14:39:00 PDT 2011


Here is the first version of the patch :)

Regards,
Iskren

On Mon, Apr 11, 2011 at 9:00 PM, Iskren Chernev <iskren.chernev at gmail.com>wrote:

> I also thought of this, but I think in one unix packet more than one
> message can be stored, which makes skipping of a single message impossible
> without the number of fds stored in the message itself.
>
> Regards,
> Iskren
>
> On Mon, Apr 11, 2011 at 8:00 PM, Bill Spitzak <spitzak at gmail.com> wrote:
>
>> Are you talking about Wayland messages between the client and compositor?
>>
>> I don't understand why you need to identify fd's in unknown message in
>> order to skip them. Isn't there a total length of the message and the fd's
>> are just embedded in this block of bytes?
>>
>> I'm probably missing something here, sorry if this is a stupid question.
>> It sounds like the client is required to do something with every fd found in
>> a message, whether they understood it or not, but that has to be wrong!
>>
>>
>> Iskren Chernev wrote:
>>
>>> Hello,
>>>
>>> I think I can do the second point in the TODO file:
>>>
>>> The message format has to include information about number of fds
>>> in the message so we can skip a message correctly.  Or we should
>>> just give up on trying to recover from unknown messages.  We need
>>> to make sure you never get a message from an interface you don't
>>> know about (using per-client id space and subscribe) or include
>>> information on number of fds, so marshalling logic can skip.
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20110413/e8f31514/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Added-number-of-fds-in-message-header.patch
Type: text/x-patch
Size: 8036 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20110413/e8f31514/attachment.bin>


More information about the wayland-devel mailing list