[Freeswitch-dev] uuid in mod_spandsp_fax.c

[Michael Collins]

*SIGH* evidently there's a magic keystroke that means "send" in GMail and I
accidentally pressed it.

As I was saying, it looks like this function is only called 6 times:

<139>:grep -n spanfax_log_message mod_spandsp_fax.c
262:void spanfax_log_message(int level, const char *msg)
716:        span_log_set_message_handler(&fax->logging,
spanfax_log_message);
717:        span_log_set_message_handler(&t30->logging,
spanfax_log_message);
772:            span_log_set_message_handler(&t38->logging,
spanfax_log_message);
773:            span_log_set_message_handler(&t30->logging,
spanfax_log_message);
827:     span_log_set_message_handler(&pvt->t38_gateway_state->logging,
spanfax_log_message);
828:     span_log_set_message_handler(&pvt->t38_core->logging,
spanfax_log_message);

I'd wager that each of those calls to the logging function would have
knowledge of the sesssion. I'd recommend adding a 3rd argument to the
logging function. In this case, I'd just add a const char *uuid and then
change each call to that function to supply the UUID as the 3rd argument.
If there is no knowledge of the session or uuid then just put an empty
string as the argument.

In any case, happy coding! :)

-MC


On Tue, Mar 27, 2012 at 1:15 AM, Anita Hall <anita.hall at simmortel.com>wrote:

> Yes Mitch, I did that but the Fax internal messages are printed by this
> line which has no knowledge of the session.
>   switch_log_printf(SWITCH_CHANNEL_LOG, fs_log_level, "%s", msg);
>
> I do not think, it can print the uuid without knowing the session ?
>
> Here is a log snippet to show uuid prefixed at other places except line
> 286 of mod_spandsp_fax.c shown above
>
> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   1...
> ....= Extension indicator: Set
> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   ....
> ..0.= Compressed/uncompressed mode: Compressed
> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   ....
> .1..= Error correction mode (ECM): ECM
> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   ....
> 0...= Frame size: 256 octets
> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   .1..
> ....= T.6 coding: Set
> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   0...
> ....= Extension indicator: Not set
> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30
> Selected compression T.6 (3)
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:327 === Negotiation Result
> =======================================================
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:328 Remote station id:
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:329 Local station id:  Sangoma Fax Ident
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:330 Transfer Rate:     4800
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:332 ECM status         on
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:333 remote country:
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:334 remote vendor:
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:335 remote model:
> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
> mod_spandsp_fax.c:337
> =========================================================================
>
>
> regards,
> Anita
>
> _________________________________________________________________________
> Professional FreeSWITCH Consulting Services:
> consulting at freeswitch.org
> http://www.freeswitchsolutions.com
>
> 
> 
>
> Official FreeSWITCH Sites
> http://www.freeswitch.org
> http://wiki.freeswitch.org
> http://www.cluecon.com
>
> FreeSWITCH-dev mailing list
> FreeSWITCH-dev at lists.freeswitch.org
> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
> http://www.freeswitch.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freeswitch.org/pipermail/freeswitch-dev/attachments/20120327/57e04ec0/attachment.html