[PATCH 3/6] libabis: ipaccess: export ipaccess_idtag_name()

[Holger Hans Peter Freyther]

On 04/07/2011 02:27 PM, pablo at gnumonks.org wrote:

Hi Pablo,

thanks for the cleaning, the criticism is to the code as it was before.

>  		t_tag = *cur++;
>  
> -		DEBUGPC(DMI, "%s='%s' ", ipac_idtag_name(t_tag), cur);
> +		DEBUGPC(DMI, "%s='%s' ", ipaccess_idtag_name(t_tag), cur);

I think we still rely that the strings on the wire have a null termination?
IIRC I once patched one version (of the many) to check for it and otherwise
discard it. Would you mind taking a look at it?


>  
> -static const char *ipac_idtag_name(int tag)
> +const char *ipaccess_idtag_name(int tag)
>  {
>  	if (tag >= ARRAY_SIZE(idtag_names))
>  		return "unknown";

we probably want to make the parameter uint?