-
For r1589.
1. file uncrustify.cpp in function process_source_list
p_file not not freed at return
2. in function do_source_file at line 1010
Variable "filename_tmp" not freed at return
3. file defines.cpp in function find_define at line 132
p_ret = (const define_tag_t *)bsearch(&tag, dl.p_tags, dl.active,
Using uninitialized value field "tag".value
4.file keywords.cpp...
2009-07-09 19:46:33 UTC in Uncrustify Code Beautifier
-
For version 0.53.
1. file brace_cleanup.cpp function push_fmr_pse
if (frm->pse_tos < (int)ARRAY_SIZE(frm->pse))
{
frm->pse_tos++;
frm->pse_tos is now 128 which is off by 1 as the array size is 128
2. indent.cpp in indent_pse_push
same thing, possible off by one for memset line 68 and line 73
3. tokenize_cleanup.cpp in function tokenize_cleanup
pc->str = "[]"; point...
2009-07-06 08:28:47 UTC in Uncrustify Code Beautifier
-
Mixing C++ and C style comments is not recommended.
There is an option to pc-lint to silence 602 error:
"fnc Nested Comments flag (default OFF).
Options: +fnc -fnc ++fnc --fnc
If this flag is ON, comments may be nested."
Tom.
2008-05-24 09:35:12 UTC in Uncrustify Code Beautifier
-
1. There is a bug in procedure for universalindent ini file creation.
Config for Uncrustify file sould be loaded first and then print_universal_indent_cfg is to be called.
sometning like this:
if (arg.Present("--universalindent"))
{
FILE *pfile = stdout;
cpd.filename = cfg_file;
if (load_option_file(cfg_file) < 0)
{
usage_exit("Unable to...
2008-04-11 14:40:11 UTC in Uncrustify Code Beautifier
-
Uncrustify 0.34 input
int foo(int a, int b)
{
if ( ! (a || b) )
{
return 1;
}
return 0;
}
after frormatting:
int foo(int a, int b)
{
if ( (!(a) || (b) ) )
{
return 1;
}
return 0;
}
!(a || b) becomes (!(a) || (b) )
Tom.
2007-06-18 19:59:50 UTC in Uncrustify Code Beautifier
-
I have just tried uncrustify on Informix ESQL/C source code. All is working fine except parsing and formating the following statement:
strcpy(demoquery1,"some SQL statement");
EXEC SQL prepare demo3id from :demoquery1;
/* the statement below is not formated correctly */
EXEC SQL declare demo3cursor cursor for demo3id;
Uncrustify reports error:
Output suffix: .uncrustify
Parsing...
2006-12-22 11:32:03 UTC in Uncrustify Code Beautifier
-
Newbie here. I just did a brief checking of uncrustify with BoundsChecker. It reported buffer a overflow in "tokenize.cpp" at line 790
while ((chunk.len > 0) &&
((chunk.str[chunk.len - 1] == ' ') ||
(chunk.str[chunk.len - 1] == '\t')))
chunk.str was "" and len was set to 2.
Tracing showed that this is a consequence of a line
538 in the same file in...
2006-12-18 22:12:05 UTC in Uncrustify Code Beautifier
