IPC: Move Log_printf after registering with Registry module
authorJuan Gutierrez <jgutierrez@ti.com>
Thu, 21 Jun 2012 17:15:26 +0000 (12:15 -0500)
committerSuman Anna <s-anna@ti.com>
Thu, 28 Jun 2012 04:48:49 +0000 (23:48 -0500)
commitc530559d45dde838a4387a2c5d7e7ee4307ee438
treed3d72135ac4a342914327627d3cbe7eebe19d343
parentecf363f09df5d60ee181f1f0833d03db710753fa
IPC: Move Log_printf after registering with Registry module

Module-specific logging support is automatic with XDC modules,
and similar functionality for non-XDC modules (like MessageQCopy)
is only possible through the Registry module. The MessageQCopy
module can have its own Diags mask, module id and logging support
functional only after registering with the Registry module (using
Registry_addModule).

Any Log_printf or other API that rely on the module id and the
descriptor, if called before Registry_addModule, will be using
invalid values for the diags mask and module id. This can cause
potential stack/memory corruption issues due to erroneous
diagnostic text lookups or other array out of bounds issues.

This patch moves the Log_printf in MessageQCopy_init only after
initialization of the Registry_Desc has been completed.

Signed-off-by: Juan Gutierrez <jgutierrez@ti.com>
Signed-off-by: Suman Anna <s-anna@ti.com>
src/ti/ipc/rpmsg/MessageQCopy.c