It seems slightly wierd to define ast_strdupa in channel.h. Shouldn't this be defined in a more generic header file? I'm only thinking from the 3rd party app perspective. If someone needs to use ast_strdupa but doesn't need the other stuff in channel.h. Just my 2 cents, Matthew