Summary: | ASTERISK-11752: Warning while compiling asterisk 1.6-beta7 | ||
Reporter: | Ovidiu Sas (ovi) | Labels: | |
Date Opened: | 2008-03-30 07:30:07 | Date Closed: | 2008-03-31 09:17:59 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Resources/res_config_sqlite |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ||
Description: | res_config_sqlite.c: In function `realtime_store_handler': res_config_sqlite.c:1208: warning: `tmp_keys' might be used uninitialized in this function res_config_sqlite.c:1208: warning: `tmp_vals' might be used uninitialized in this function ****** ADDITIONAL INFORMATION ****** tmp_keys and tmp_vals are initialized inside the for loop and this is confusing the compiler. The previous check on params_count makes sure that we are entering the loop and these values will be properly initialized. However, the values should be initialized (just like tmp_keys2 and tmp_vals2) to get rid of this warning. | ||
Comments: | By: Digium Subversion (svnbot) 2008-03-31 09:15:59 Repository: asterisk Revision: 111961 U trunk/res/res_config_sqlite.c ------------------------------------------------------------------------ r111961 | file | 2008-03-31 09:15:57 -0500 (Mon, 31 Mar 2008) | 4 lines Initialize all these here tmp pointers at declaration. They confused some compilers a wee bit. (closes issue ASTERISK-11752) Reported by: ovi ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=111961 By: Digium Subversion (svnbot) 2008-03-31 09:17:59 Repository: asterisk Revision: 111962 _U branches/1.6.0/ U branches/1.6.0/res/res_config_sqlite.c ------------------------------------------------------------------------ r111962 | file | 2008-03-31 09:17:59 -0500 (Mon, 31 Mar 2008) | 12 lines Merged revisions 111961 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r111961 | file | 2008-03-31 11:20:39 -0300 (Mon, 31 Mar 2008) | 4 lines Initialize all these here tmp pointers at declaration. They confused some compilers a wee bit. (closes issue ASTERISK-11752) Reported by: ovi ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=111962 |