Hi Brian, On Fri, Feb 18, 2011, Brian Gix wrote: > Example Included Service Attribute missing UUID16 field. > MTU Exchange not bounded by actual size required not to crash. > --- > attrib/example.c | 3 ++- > src/attrib-server.c | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) Even though these are some quite trivial changes, you essentially have to independent fixes here and they should therefore be in two separate patches. Also make sure that the summary line of each patch gives at least some idea of what is being fixed. The way you formulated it now just tells us that "something" found at UPF38 was fixed. Johan P.S. I think you can stop adding extra people to CC in your patch submissions. --to linux-bluetooth@xxxxxxxxxxxxxxx should be enough. -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html