> -----Original Message----- > From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media- > owner@xxxxxxxxxxxxxxx] On Behalf Of Andreas Oberritter > Sent: jeudi 19 mai 2011 14:58 > To: Tomer Barletz > Cc: Brice DUBOST; linux-media@xxxxxxxxxxxxxxx > Subject: Re: [libdvben50221] [PATCH] Assign same resource_id in > open_session_response when "resource non-existent" > > On 05/18/2011 09:16 PM, Tomer Barletz wrote: > > On Tue, May 17, 2011 at 8:46 AM, Brice DUBOST <braice@xxxxxxxxxx> > wrote: > >> On 18/01/2011 15:42, Tomer Barletz wrote: > >>> Attached a patch for a bug in the lookup_callback function, were in > >>> case of a non-existent resource, the connected_resource_id is not > >>> initialized and then used in the open_session_response call of the > >>> session layer. > >>> > >> > >> Hello > >> > >> Can you explain what kind of bug it fixes ? > >> > >> Thanks > >> > > > > The standard states that in case the module can't provide the > > requested resource , it should reply with the same resource id - this > > is the only line that was added. > > Also, since the caller to this function might use the variable > > returned, this variable must be initialized. > > The attached patch solves both bugs. > > Can you please resend the patch inline with a proper signed-off-by line, > in order to get it tracked by patchwork.kernel.org? > Yes, of course, but I don't find information that can help me to provide the correct format. Is-there a documentation somewhere that explains how patches must be formatted to be correctly tracked? > Regards, > Andreas > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" > in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo > info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
