diff options
author | rizzo <rizzo@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-07-21 17:32:00 +0000 |
---|---|---|
committer | rizzo <rizzo@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-07-21 17:32:00 +0000 |
commit | 340ad4be77de77619d8b28540228148260c6ed53 (patch) | |
tree | fa4a846831be4a600d98654ca696dc6f16e9d707 | |
parent | 712969305d814a0e17e2caee39034ab3a6d08845 (diff) |
Add a note to document how the temporary 'pvt' should be initialized
before using it.
I am unclear on the details right now so i hope someone can comment
more. The obvious (and lazy) approach would be to bzero() all of it
(except for the string pool), but isn't that too much work ?
Feedback wanted here...
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@76313 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | channels/chan_sip.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 773f34004..02a8e62ce 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -6353,6 +6353,13 @@ static int transmit_response_using_temp(ast_string_field callid, struct sockaddr return -1; } + /* XXX the structure may be dirty from previous usage. + * Here we should state clearly how we should reinitialize it + * before using it. + * E.g. certainly the threadstorage should be left alone, + * but other thihngs such as flags etc. maybe need cleanup ? + */ + /* Initialize the bare minimum */ p->method = intended_method; |