Hi,
Dereferencing a potential null pointer (ih_array), lines 122, 116.
--- \dll\iup\a\src\iup_dlglist.cThu Jul 25 16:27:06 2019
+++ iup_dlglist.c Mon Nov 04 13:47:53 2019
@@ -115,7 +115,7 @@
int i = 0, count;
Ihandle** ih_array = (Ihandle**)malloc(idlg_count *
Hi,
Dererencing a potential null pointer at line 294.
--- \dll\iup\a\src\iup_attrib.c Thu Oct 03 21:48:51 2019
+++ iup_attrib.cMon Nov 04 13:51:57 2019
@@ -287,12 +287,12 @@
/* store the names before updating so we can add or remove attributes during
the update */
name =
good test is to to copy that command line, and manually run it in a
regular prompt.
Best,
Scuri
Em seg, 4 de nov de 2019 12:28, Ranier VF
mailto:ranier_...@hotmail.com>> escreveu:
Hi Scuri,
"Or the shortcut "Developer Command Prompt for VS 2017"."
Yes this, this is being
obj/dll15/iup_array.obj] Error 2
makefile:12: recipe for target 'iup' failed
win-make[1]: *** [iup] Error 2
makefile:9: recipe for target 'iup' failed
win-make: *** [iup] Error 2
E:\dll\iup\iup>
____
De: Ranier VF
Enviado: segunda-feira, 4 de novembro de 2019
est,
Scuri
Em seg., 4 de nov. de 2019 às 10:31, Ranier VF
mailto:ranier_...@hotmail.com>> escreveu:
Hi,
Some progress, after creating manually all obj and lib.
E:\dll\iup\iup>win-make --debug
GNU Make 4.1
Built for i686-w64-mingw32
Copyright (C) 1988-2014 Free Software Foundation,
rget 'iup' failed
win-make[1]: *** [iup] Error 2
makefile:9: recipe for target 'iup' failed
win-make: *** [iup] Error 2
File iup_array.c is under src directory, but makefile do not fill the src dir,
why?
Thanks.
Ranier Vilela
________
De: Ranier VF
Enviado: segunda-fe
iup>
____
De: Ranier VF
Enviado: segunda-feira, 4 de novembro de 2019 11:22
Para: IUP discussion list.
Assunto: [Iup-users] IUP Compilation with msvc 2017
Hi Scuri,
Windows 64 bits
msvc 2017
set TEC_UNAME=dll15
set VC15=c:/progra~2/micros~2/2017/community/
Hi Scuri,
Windows 64 bits
msvc 2017
set TEC_UNAME=dll15
set VC15=c:/progra~2/micros~2/2017/community/vc
set NO_DEPEND=Yes
''; echo 'Tecmake: Starting [ iup:dll15 ]'
if [ ! -d ../obj/dll15 ] ; then mkdir -p ../obj/dll15 ; fi
! foi inesperado neste momento.
../tecmakewin.mak:1662: recipe for
Hi Scuri,
Can be for the msvc 2017 compiler (64 bits)
Thanks.
Ranier Vilela
De: Ranier VF
Enviado: sexta-feira, 1 de novembro de 2019 18:02
Para: IUP discussion list.
Assunto: Re: [Iup-users] IUP Library with Mingw64
Hi,
Windows 7 64 bits
gcc (GCC
also help
you.
The first thing to check if the mingw version is the same (or nearest) as the
one I use to build the pre-compiled binaries.
Best,
Scuri
Em sex, 1 de nov de 2019 às 13:17, Ranier VF
mailto:ranier_...@hotmail.com>> escreveu:
Hi Scuri,
I'm having problems with compilation
Hi Scuri,
I'm having problems with compilation IUP with Mingw64.
How to make IUP library with Mingw64 with Windows 7 env?
Can you show the correctly setup vars?
Thank you.
Ranier Vilela
___
Iup-users mailing list
Iup-users@lists.sourceforge.net
Hi,
Valuable message.
For my part, what I have to say is, use all resources to deal with bugs
in C development, because is easy make new bugs and hard to find us.
With respect "misleading indentation", I follow K here, example:
if {
a = 1;
} else {
a = 2;
}
The explicit use { } is much better
Hi,
Very powerful report, but, unfortunately this being ignored.
Most is in 3rd party libraries and IUP team already showed
that there is nothing to do.
Would be better, direct users this 3rd libraries, forward this reports
to own mainteners, future versions can have this fixed and
incorpored
Hi, thank you.
Apologies to everyone if was rude.
I am a C developer, and I love C, because is most powerful language
in my opinion. In my own code, I try do write the best C code possible,
following the masters K (The Bests).
So when I see the C being underused, I get frustrated.
Examples:
Hi,
Ok, this is a little test.
Very well, wich we can confirm, that priority of IUP team is not
improve iup code, but, only add new features.
So why spend time and energy, if no one has interest...
Thank you.
Best regards,
Ranier Vilela
De: Ranier VF
Hi,
Here some performance optimize in function winButtonWmCommand.
The "_IUPBUT_INSIDE_ACTION" is not used anywhere else and can
changed by much faster C resource.
--- \dll\iup\a\src\win\iupwin_button.c Mon Jun 03 20:25:39 2019
+++ iupwin_button.c Wed Jun 05 10:58:31 2019
@@ -660,21 +660,17
Hi,
Here some warnings coming from msvc compiler:
c:\dll\iup\iup\srcmglplot\src\pde.cpp(221) : warning C4701: potentially
uninitialized local variable 'ham' used
c:\dll\iup\iup\srcmglplot\src\prim.cpp(403) : warning C4701: potentially
uninitialized local variable 'n2' used
re careful. But that's not the case.
Best,
Scuri
Em seg, 3 de jun de 2019 às 23:05, Ranier VF
mailto:ranier_...@hotmail.com>> escreveu:
Hi,
Several fixes for the iupwin_webbrowser.cpp
--- \dll\iup\a\srcweb\iupwin_webbrowser.cpp Wed Apr 03 14:12:26 2019
+++ iupwin_webbrowser.cpp
ID 210668 (#1 of 1): Explicit null dereferenced
(FORWARD_NULL)
The realloc function does the malloc job. And it will only increase the size
of the value pointer when necessary.
Best,
Scuri
Em seg, 3 de jun de 2019 às 22:29, Ranier VF
mailto:ranier_...@hotmail.com>> escreveu:
Hi,
Sorry
Hi,
Sorry but can't undestand the logic.
iupmatex_clipboard.c
line 505, call function:
value = iMatrixExStrCopyData(value, _max_size, data, value_len);
iMatrixExStrCopyData, call memcpy with value without malloc?
static char* iMatrixExStrCopyData(char* value, int *value_max_size, const char*
Hi Scuri,
Thank you for the commits.
However I have some comment about the commit 5211:
https://sourceforge.net/p/iup/iup/5211/
While comments like /* ignore return value */ is great for
programmers, for the Coverity and GCC do not mean anything.
The correct way to silence warnings coming is:
of 1): Division or modulo by zero
(DIVIDE_BY_ZERO)
If that function is called with max ==0 then it is a much more serious error
than a divide by zero, and we would mascarade the error by adding such code.
Best,
Scuri
Em sáb, 25 de mai de 2019 às 08:54, Ranier VF
mailto:ranier_
Hi,
This patch try fixes :
1. Coverity and GCC report about absence break into switch or implicit
fallthrough:
https://stackoverflow.com/questions/45129741/gcc-7-wimplicit-fallthrough-warnings-and-portable-way-to-clear-them
https://en.wiktionary.org/wiki/fall-through
[Iup-users] CID 210676 (#1 of 1): Resource leak (RESOURCE_LEAK)
The code you added is an unreachable code. The "for" never ends normally, only
in the return.
Em sáb, 25 de mai de 2019 às 08:28, Ranier VF
mailto:ranier_...@hotmail.com>> escreveu:
Hi,
Fix RESOURCE_LEAK at iup
Hi,
msvc 2010 compiler warnings:
iupPlotDraw.cpp(322) : warning C4061: enumerator 'IUP_PLOT_XY' in switch of
enum 'iupPlotLegendPosition' is not explicitly handled by a case label
c:\dll\iup\iup\srcplot\iupPlot.h(21) : see declaration of
'iupPlotLegendPosition'
iupPlotDraw.cpp(322) :
Hi,
When compiling iup_calendar.c in windows port, var NOT_WORKING is
not defined;
What is the correct definition?
Best regards,
Ranier Vilela
___
Iup-users mailing list
Iup-users@lists.sourceforge.net
Hi,
Multiple fixes in iupwin_newfiledlg.c:
--- ..\..\a\srcfiledlg\iupwin_newfiledlg.cppFri Sep 14 17:33:33 2018
+++ iupwin_newfiledlg.cpp Sat May 25 10:43:31 2019
@@ -334,48 +334,25 @@
COMDLG_FILTERSPEC* filters =
(COMDLG_FILTERSPEC*)malloc((buffSize)*sizeof(COMDLG_FILTERSPEC));
Hi,
Should be return?
Or is better check return value?
--- \dll\iup\a\srccontrols\matrix\iupmat_key.c Mon Dec 11 14:52:46 2017
+++ iupmat_key.cSat May 25 10:26:47 2019
@@ -173,7 +173,7 @@
case K_CR:
if (!ih->data->edit_hide_onfocus && ih->data->editing)
-
Hi,
--- \dll\iup\a\src\win\iupwindows_info.cThu Apr 19 08:22:24 2018
+++ iupwindows_info.c Sat May 25 10:21:08 2019
@@ -87,7 +87,7 @@
sprintf(str, "%d.%d.%d", (int)osvi.dwMajorVersion, (int)osvi.dwMinorVersion,
(int)osvi.dwBuildNumber);
/* Display service pack (if any). */
- if
Hi,
--- \dll\iup\a\src\win\iupwin_toggle.c Mon Aug 20 06:57:56 2018
+++ iupwin_toggle.c Sat May 25 10:17:01 2019
@@ -578,6 +578,7 @@
case BN_DOUBLECLICKED:
if (iupAttribGetBoolean(ih, "IGNOREDOUBLECLICK"))
return 0;
+/* FALLTHROUGH */
case BN_CLICKED:
{
Hi,
--- \dll\iup\a\src\win\iupwin_label.c Tue Nov 27 17:25:24 2018
+++ iupwin_label.c Sat May 25 10:09:27 2019
@@ -44,7 +44,7 @@
int xpad = ih->data->horiz_padding,
ypad = ih->data->vert_padding;
int x, y, width, height, bpp;
- HBITMAP hBitmap, hMask = NULL;
+ HBITMAP
Hi,
iup_str.c:
mm('®')= 96; mm('°')= 97; mm('µ')= 98; mm('¶')= 99; mm('…')=100;
mm('†')=101; mm('‡')=102; mm('•')=103; mm('•')=104; mm('‰')=105; mm('0')=106;
mm('¼')=107; mm('½')=108; mm('¾')=109; mm('1')=110; mm('¹')=111;
mm('•')=103; mm('•')=104; /* value_overwrite: Overwriting previous
Hi,
--- \dll\iup\a\src\iup_globalsdlg.c Tue Apr 23 18:07:31 2019
+++ iup_globalsdlg.cSat May 25 09:56:52 2019
@@ -216,9 +216,8 @@
IupSetAttributeId(list2, "", i + 1, attr_names[i]);
IupSetIntId(list2, "_IUP_READONLY", i + 1, 0);
}
-
-free(attr_names);
}
+
Hi,
--- \dll\iup\a\src\iup_expander.c Mon Oct 29 14:53:02 2018
+++ iup_expander.c Sat May 25 09:49:19 2019
@@ -384,9 +384,10 @@
int frame_time = iupAttribGetInt(ih, "FRAMETIME");
int num_frames = iupAttribGetInt(ih, "NUMFRAMES");
int time_delay = iupAttribGetInt(animate_timer,
Hi,
Fix iupwin_dragdrop.c:
--- \dll\iup\a\src\win\iupwin_dragdrop.cWed Apr 24 17:12:36 2019
+++ iupwin_dragdrop.c Sat May 25 09:34:29 2019
@@ -391,7 +391,7 @@
{
IFns cbDragDataSize;
IFnsVi cbDragData;
- int size;
+ int size = 0;
void *pData;
TCHAR type[256];
@@ -405,7
Hi,
--- ..\..\a\srcfiledlg\iupwin_newfiledlg.cppFri Sep 14 17:33:33 2018
+++ iupwin_newfiledlg.cpp Sat May 25 09:09:00 2019
@@ -719,7 +719,7 @@
psiResult->Release();
}
}
-else
+else if (opfd != NULL)
{
IShellItemArray *psiaResult;
Best regards,
Hi,
Fix iup_flatscrollbar.c:
--- ..\..\a\src\iup_flatscrollbar.c Mon Jan 28 18:13:06 2019
+++ iup_flatscrollbar.c Sat May 25 08:53:03 2019
@@ -138,14 +138,19 @@
static void iFlatScrollBarCalcHandler(int size, int arrow_size, int max, int
d, int sb_size, int pos, int* pos1, int* pos2)
{
Hi,
Fix iup_globalattrib.c:
--- ..\..\a\src\iup_globalattrib.c Tue Apr 23 14:33:33 2019
+++ iup_globalattrib.c Sat May 25 08:42:08 2019
@@ -133,7 +133,7 @@
iupdrvWarpPointer(x, y);
return;
}
- if (iupStrEqual(name, "MOUSEBUTTON"))
+ if (value != NULL && iupStrEqual(name,
Hi,
Fix RESOURCE_LEAK at iup_attrib.c
--- ..\..\a\src\iup_attrib.cSat Jan 26 16:02:13 2019
+++ iup_attrib.cSat May 25 08:25:17 2019
@@ -1499,6 +1499,11 @@
break;
}
}
+ if (name)
+ {
+ IupStoreAttribute(ih, name, value);
+ free(name);
+ }
}
Ihandle*
Hi,
This a new report by Coverity about IUP 3.27 Windows port.
Best regards,
Ranier Vilela
De: scan-ad...@coverity.com
Enviado: sábado, 25 de maio de 2019 03:02
Para: ranier_...@hotmail.com
Assunto: New Defects reported by Coverity Scan for IUP - Portable
Hi,
I'm around with a IUP bug. Below show the callstack:
may be wrong.
1288cfb0 0111 006a iup!iupObjectCheck+0x8
1288cfb0 0111 006a iup!iupdrvImageDestroy+0x2dbf
000303de 0111 006a iup!iupwinBaseMsgProc+0x61f
03df3160 000303de 0111 USER32!GetDC+0x6d
03df3160
Hi Scuri,
Thank you for the release.
Tested and works fine!
Best regards,
Ranier Vilela
De: Antonio Scuri
Enviado: terça-feira, 30 de abril de 2019 20:19
Para: IUP discussion list.
Assunto: [Iup-users] IUP 3.27
Hi All,
We just released IUP version
?
It was one fix that exposed another bug.
Just fixed this other bug and committed to SVN.
Best,
Scuri
Em ter, 15 de jan de 2019 às 14:43, Ranier VF
mailto:ranier_...@hotmail.com>> escreveu:
Hi,
Maybe this is expected behavior or be a regression.
txt_medium_size = 14;
IupSetInt(mat
Hi,
Maybe this is expected behavior or be a regression.
txt_medium_size = 14;
IupSetInt(mat_items, "FONTSIZE", txt_medium_size);
IupMatrix with IUP-3.26, all fonts are small.
IupMatrix with IUP-3.25, all fonts are big.
IUP-3.26, IupMatrix ignores FONTSIZE attribute or fontsize=14,
is
Hi,
I'm using callback VALUECHANGED_CB with list control, to populate
another list control, when user selects one item. Inside function, I use
VALUE attribute to know what item was selected.
But, my list controls don´t allow edition, or, EDITBOX=NO.
If this can help you, I use too AttributeHandle
Hi,
Maybe this can help you.
https://docs.microsoft.com/en-us/windows/desktop/controls/cookbook-overview
Best regards.
Ranier
De: Johann Philippe via Iup-users
Enviado: domingo, 25 de novembro de 2018 17:59
Para: Antonio Scuri
Cc: Johann Philippe; IUP
Hi,
IUP 3.25
IupDatePick ignores solemnly READONLY=YES attribute.
Best regards,
Ranier Vilela.
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org!
Hi Scuri,
IupGetParam("Test", NULL, 0, "Number: %i\n", );
Always show the dialog with 0 filled, this disrupts usability.
Aways user must be delete 0 to digit correct number.
Is possible change behavior IupGetParam to start with empty value?
Best regards,
Ranier Vilela
Hi,
Solved finally, heap corrupted.
Howto prevent:
ASSERT(strlen(buf) < sizeof(buf);
Best regards,
Ranier Vilela
De: Ranier VF
Enviado: segunda-feira, 23 de julho de 2018 17:06
Para: IUP discussion list.
Assunto: [Iup-users] IUP crash with release
Hi,
I have dealing with a insidius bug in my app. If somebody can help, I thank you
so much.
This line:
IupSetStrAttributeId2(mat, "", nrows, 0, item_code);
run normally with DEBUG version (msvc 2010), but with RELEASE version, crash!
This exception may be expected and handled.
Hi,
Thanks for a clear explanation and simple solution for the problem.
Best regards,
Ranier Vilela
De: Theron
Enviado: quarta-feira, 18 de julho de 2018 21:03
Para: iup-users@lists.sourceforge.net
Assunto: Re: [Iup-users] Use utf-8 source encoding rather
Hi Scuri,
Great news, thank you for the new release IUP 3.25.
Most GDI USAGE ERRORS, is gone, remain 2 only.
Error #6: GDI USAGE ERROR: free mismatch for DC 0x9d01119f: use ReleaseDC only
for GetDC and DeleteDC only for CreateDC
Error #7: GDI USAGE ERROR: free mismatch for DC 0x44010fcd: use
Hi,
"Please ignore anyone who says "I don't need it so you can't have it" - but,
obviously, it has to be a
solution that causes them no pain when/if it does go in, and your proposal
sounds just like that to me."
I do not know if it was addressed to me. But I will answer anyway.
I not agaist
Hi,
"IUP should not be the solution to every problem. It would be like
incorporating a cross-platform C-runtime into IUP when that is the wrong
approach. IUP is great as it is and for specialized problems it will never be
able to beat a mature third party app, and right now my favorite app for the
Hi,
"This proposal is a tiny patch to IUP, one that nobody except the
implementors will notice. It does not break current behavior or create
any trade offs. But it solves real problems deal with the fact that
the real world uses threads, even inside the native platforms. If you
don't have this
Hi,
"I think we're in agreement here. The idea of "IupPostMessage",
inspired from Windows PostMessage solves this problem elegantly. The
implementation specific ways to pull this off on each platform are
hidden from the user perspective."
Great.
"From my "Idea 2" proposal, what I'm suggesting
Hi,
I strong believe thar PostMessage solves the request: "update gui, outside
main thread"
Native handles, like Windows Handle, is hidden from the user perspective.
Therefore using in Windows driver and not using in GTK, IOS and Android
drivers, will not disturb users.
In your example, You
Hi,
I strong believe thar PostMessage solves the request: "update gui, outside
main thread"
Native handles, like Windows Handle, is hidden from the user perspective.
Therefore using in Windows driver and not using in GTK, IOS and Android
drivers, will not disturb users.
In your example, You
Hi,
Obviously I'm not saying to Call PostMessage Windows function,
in other platforms.
So I understood, only have problem is Windows Loop Event (IUP),
and the autor what do you cite, say clearly that PostThreadMessage
is wrong way to solve this (update GUI in another thread).
Best,
Ranier
Hi,
"But in when using IUP, we can't just reasonably call Window's
PostMessage because it is not cross-platform. It won't work on any
other platform except Windows.
This whole proposal is on how to make something like PostMessage
cross-platform."
It's OK, but in your app example, I dont see any
Hi,
"This is a proposal to introduce a way to post and run events on the
main UI thread."
Just my two cents too.
All this is related with GUI update, in another thread, other the main thread.
Then other types updates, should not interfere with IUP.
IUP Dialog have a WID (windows HWND). Why not
Hi,
"This is a proposal to introduce a way to post and run events on the
main UI thread."
Just my two cents too.
All this is related with GUI update, in another thread, other the main thread.
Then other types updates, should not interfere with IUP.
IUP Dialog have a WID (windows HWND). Why not
Hi,
If I understood, sur-behoffski, wants to know, howto
pass parameter "-p" from tecmake to make.
Best.
Ranier Vilela
De: Antonio Scuri
Enviado: quinta-feira, 1 de março de 2018 13:51
Para: IUP discussion list.
Assunto: Re:
is ok not to
release those resources. When the application is closed they will be
automatically deleted by the system.
Best,
Scuri
2018-02-20 9:54 GMT-03:00 Ranier VF
<ranier_...@hotmail.com<mailto:ranier_...@hotmail.com>>:
Hi,
In relation this bug, some consideration?
Best rega
you using?
Best,
Scuri
2018-02-10 22:58 GMT-02:00 Ranier VF
<ranier_...@hotmail.com<mailto:ranier_...@hotmail.com>>:
Hi,
Clearly documents says TVM_GETITEM message can fail.
https://msdn.microsoft.com/en-us/library/windows/desktop/bb773596(v=vs.85).aspx
Iupwin_tree.c there are lot
stDestroyAll optimized version
Yes, because when a dialog is destroyed the idlglist list is updated, so
list->next will be invalid.
Best,
Scuri
2018-02-13 11:13 GMT-02:00 Ranier VF
<ranier_...@hotmail.com<mailto:ranier_...@hotmail.com>>:
Hi,
Is there really any special reason,
Hi,
In relation this bug, some consideration?
Best regards,
Ranier Vilela
De: Ranier VF <ranier_...@hotmail.com>
Enviado: segunda-feira, 29 de janeiro de 2018 12:01
Para: IUP discussion list.
Assunto: Re: [Iup-users] IUP 3.24 RESOURCE_LEAK (iupwin_
Hi,
Is there really any special reason, for what
iupDlgListDestroyAll do not be like that?
--- a\src\iup_dlglist.c Tue Feb 13 11:05:14 2018
+++ b\src\iup_dlglist.c Tue Feb 13 11:11:25 2018
@@ -112,24 +112,9 @@
void iupDlgListDestroyAll(void)
{
- int i = 0, count;
- Ihandle** ih_array =
Hi
Nice work.
Improvemnts in build iup, are always good.
Best,
Ranier Vilela
De: sur-behoffski
Enviado: terça-feira, 30 de janeiro de 2018 12:32
Para: IUP discussion list.
Assunto: [Iup-users] Getting closer to my IM/CD/IUP
n list.
Assunto: Re: [Iup-users] IUP 3.24 RESOURCE_LEAK (iupwin_menu.c)
Those DeleteObject(hBitmap*) can not be called. They are stored in a cache
inside the IupImage, they will be released later when the image is destroyed.
Best,
Scuri
2018-01-24 22:30 GMT-02:00 Ranier VF
<ranier_...@ho
Hi,
Issue found by Coverity Scan.
CID 259207 (#1 of 1): Dereference null return value (NULL_RETURNS)
4. dereference: Dereferencing a null pointer ih.
--- a\src\win\iupwin_tabs.c Mon Dec 11 14:52:42 2017
+++ b\src\win\iupwin_tabs.c Wed Jan 24 23:11:36 2018
@@ -436,11 +436,15 @@
{
handle */
Ihandle* dlg = iupDlgListFirst();
-do
+while(dlg)
{
if (dlg->handle)
{
@@ -220,7 +220,7 @@
break;
}
dlg = iupDlgListNext();
-} while (dlg);
+}
}
De: Ranier VF <r
Hi,
Issue found by Coverity Scan.
CID 259174 (#1 of 1): Logically dead code (DEADCODE)
--- a\src\win\iupwin_loop.c Mon Dec 11 14:52:42 2017
+++ b\src\win\iupwin_loop.c Wed Jan 24 23:04:34 2018
@@ -113,7 +113,6 @@
}
} while (ret);
- win_main_loop--;
return IUP_NOERROR;
}
return 1;
}
+break;
case WM_SETFOCUS:
case WM_KILLFOCUS:
case WM_MOUSELEAVE:
De: Ranier VF <ranier_...@hotmail.com>
Enviado: quinta-feira, 25 de janeiro de 2018 00:35
Para: IUP discussion list.
Assunto: [Iup-users] IUP 3.24 RESOURCE LEAK (iup
Hi,
Issue found by Coverity Scan.
CID 259200 (#4 of 4): Dereference null return value (NULL_RETURNS)
dereference: Dereferencing a null pointer cb.
--- a\src\win\iupwin_filedlg.c Mon Dec 11 14:52:42 2017
+++ b\src\win\iupwin_filedlg.c Wed Jan 24 22:56:29 2018
@@ -548,7 +548,10 @@
Ihandle*
Hi,
Issue found by Coverity Scan.
CID 259179 (#1 of 1): Missing break in switch (MISSING_BREAK)
unterminated_case: The case for value 276U is not terminated by a 'break'
statement.
--- a\src\win\iupwin_canvas.c Tue Jan 16 20:23:32 2018
+++ b\src\win\iupwin_canvas.c Wed Jan 24 22:40:51 2018
Hi,
Issue found by Coverity Scan.
CID 259212 (#1 of 1): Resource leak (RESOURCE_LEAK)
4. leaked_handle: Handle variable hBitmap going out of scope leaks the handle.
--- a\src\win\iupwin_list.c Mon Dec 11 14:52:42 2017
+++ b\src\win\iupwin_list.c Wed Jan 24 22:35:04 2018
@@ -962,12 +962,14
Hi,
Issue found by Coverity Scan.
CID 259189 (#1-2 of 2): Resource leak (RESOURCE_LEAK)
--- a\src\win\iupwin_menu.c Mon Dec 11 14:52:42 2017
+++ b\src\win\iupwin_menu.c Wed Jan 24 22:26:33 2018
@@ -431,9 +431,13 @@
hBitmapChecked = hBitmapUnchecked;
Hi Scuri,
I'm compiling IM, CD and IUP in Windows env.
I have found issues with tecmakewin.mak
1) the files .wdep in IM and CD, are created with size of 5 and content "" or
are not being created.
solution: copy content file im.wdep with size of 1 byte to replace
im_process.wdep,
for example.So,
-02:00 Ranier VF
<ranier_...@hotmail.com<mailto:ranier_...@hotmail.com>>:
Hi,
Very disappointing!
In world Open Source, the least that is expected is recognition.
No mention of my name and none mention about bugs fixeds in IUP.
When asked, some time ago, Scuri say contribute with code.
C
Hi,
Very disappointing!
In world Open Source, the least that is expected is recognition.
No mention of my name and none mention about bugs fixeds in IUP.
When asked, some time ago, Scuri say contribute with code.
Coverity Team asked me, proof of colaboration to release report.
They found nothing
third report was a serious bug... Thanks.
Fixed and committed to SVN.
Best,
Scuri
2018-01-19 16:20 GMT-02:00 Ranier VF
<ranier_...@hotmail.com<mailto:ranier_...@hotmail.com>>:
Hi,
Multiple issues found by Coverity Scan.
CID 210502 (#1 of 1): Division or modulo by zero (DIVIDE_BY_ZE
Hi,
Multiple issues found by Coverity Scan.
CID 210502 (#1 of 1): Division or modulo by zero (DIVIDE_BY_ZERO)
CID 210590 (#1 of 1): Infinite loop (INFINITE_LOOP)
CID 210665 (#1 of 1): Division or modulo by zero (DIVIDE_BY_ZERO)
--- a\srcscintilla\iup_scintilladlg.c Fri Jan 19 16:00:51 2018
+++
Hi,
Issue found by Coverity Scan.
CID 210500 (#1 of 1): Incorrect sizeof expression (BAD_SIZEOF)
bad_sizeof: Taking the size of parameter list is suspicious because although it
is declared as an array, it is actually a pointer. Sizeof will return the size
of a pointer, which may not be intended.
Hi,
Issue found by Coverity Scan.
CID 210522 (#1 of 1): Unused value (UNUSED_VALUE)
assigned_value: Assigning value from temp to state here, but that stored value
is overwritten before it can be used
file: iup_maskmatch.c
line: 482
state = temp;
Best.
Ranier
Hi,
Issue found by Coverity Scan.
CID 210632 (#1 of 1): Logically dead code (DEADCODE)
array_null: Comparing an array to null is not useful: ilex.name, since the test
will always evaluate as true.
--- a\src\iup_ledlex.c Sun Jul 26 03:10:06 2015
+++ b\src\iup_ledlex.c Fri Jan 19 15:20:27 2018
!iupStrEqualNoCase(afunc->default_value, afunc->system_default)) ||
If they are not equal OR
then they are equal and not NULL at:
(afunc->call_global_default &&
iupGlobalDefaultColorChanged(afunc->default_value)))
There is no need for " && afunc->default
y zero
(DIVIDE_BY_ZERO)
If sb_range=0 then sb_ymax=sb_ymin then the scrollbar is not shown, then this
code is not called.
Best,
Scuri
2018-01-19 0:29 GMT-02:00 Ranier VF
<ranier_...@hotmail.com<mailto:ranier_...@hotmail.com>>:
Hi,
Issue found by Coverity Scan.
CID 210649 (#2-4 of 4):
Hi,
Issue found by Coverity Scan.
CID 210649 (#2-4 of 4): Division or modulo by zero (DIVIDE_BY_ZERO)
divide_by_zero: In expression (start_pos - sb_ymin) * range / sb_range,
division by expression sb_range which may be zero has undefined behavior.
--- a\srcglcontrols\iup_glscrollbars.c Wed Apr
Hi,
Issue found by Coverity Scan.
CID 210629 (#1 of 1): Argument cannot be negative (NEGATIVE_RETURNS)
return_negative_constant: Explicitly returning negative value -1.
iup_childtree.c, function IupGetChildCount:
--- a\src\iup_childtree.c Tue May 02 23:06:40 2017
+++ b\src\iup_childtree.c
Hi,
Issue found by Coverity Scan.
CID 210573 (#1 of 1): Dereference after null check (FORWARD_NULL)
var_deref_model: Passing null pointer afunc->default_value to
iupGlobalDefaultColorChanged, which dereferences it.
--- a\src\iup_classattrib.c Fri Oct 14 13:48:50 2016
+++
Hi,
Issue found by Coverity Scan.
CID 210637 (#2 of 2): Dereference null return value (NULL_RETURNS)
dereference: Dereferencing a null pointer cb.
--- a\src\gtk\iupgtk_filedlg.c Fri Jun 09 13:26:00 2017
+++ b\src\gtk\iupgtk_filedlg.c Thu Jan 18 15:42:06 2018
@@ -234,13 +234,16 @@
/*
Hi,
Issue found by Coverity Scan.
CID 210622 (#1 of 1): Resource leak (RESOURCE_LEAK)
--- a\src\gtk\iupgtk_colordlg.c Sun Jul 26 03:10:04 2015
+++ b\src\gtk\iupgtk_colordlg.c Wed Jan 17 23:12:37 2018
@@ -48,6 +48,7 @@
}
palette = palette_p;
}
+ free(gtk_str);
str[off-1] = 0; /*
D_NULL)
In that case the loop will be:
while (capture != NULL)
capture = capture->next_one;
It is not necessarily an infinite loop.
Best,
Scuri
2018-01-17 19:48 GMT-02:00 Ranier VF
<ranier_...@hotmail.com<mailto:ranier_...@hotmail.com>>:
Hi,
With fix in th SVN, th
ra, 17 de janeiro de 2018 21:04
Para: IUP discussion list.
Assunto: Re: [Iup-users] CID 210623 (#1 of 1): Explicit null dereferenced
(FORWARD_NULL)
Fixed and committed to SVN. I have no idea if that condition could occur...
Best,
Scuri
2018-01-16 23:29 GMT-02:00 Ranier VF
<ranier_...
Hi,
Issue found by Coverity Scan.
CID 210578 (#1 of 1): Dereference null return value (NULL_RETURNS)
--- a\srccontrols\matrix\iupmat_getset.cTue Sep 19 12:43:20 2017
+++ b\srccontrols\matrix\iupmat_getset.cTue Jan 16 23:52:24 2018
@@ -138,7 +138,14 @@
{
/* only called in
Hi,
Issue found by Coverity Scan.
CID 210583 (#1 of 1): Division or modulo by zero (DIVIDE_BY_ZERO)
. divide_by_zero: In expression time_delay / frame_time, division by expression
frame_time which may be zero has undefined behavior.
--- a\src\iup_expander.cMon Sep 04 15:31:16 2017
+++
Hi,
Issue found by Coverity Scan.
CID 210616 (#1 of 1): Missing varargs init or cleanup (VARARGS)
--- a\src\iup_scanf.c Sun Jul 26 03:10:04 2015
+++ b\src\iup_scanf.c Tue Jan 16 23:38:37 2018
@@ -175,9 +175,9 @@
}
++fields_out_count;
}
- va_end(va);
cleanup:
+ va_end(va);
Hi,
Issue found by Coverity Scan.
CID 210621 (#1 of 1): Copy-paste error (COPY_PASTE_ERROR)
copy_paste_error: max_w in max_w > geometry.min_height looks like a copy-paste
error.
--- a\src\gtk\iupgtk_dialog.c Wed Oct 11 19:38:34 2017
+++ b\src\gtk\iupgtk_dialog.c Tue Jan 16 23:33:23 2018
@@
Hi,
Issue found by Coverity Scan.
CID 210638 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)
--- a\srcglcontrols\iup_gldraw.cThu Aug 04 14:58:16 2016
+++ b\srcglcontrols\iup_gldraw.cTue Jan 16 23:17:34 2018
@@ -571,7 +570,7 @@
glTranslated(0.0, (double)-lineheight, 0.0);
1 - 100 of 172 matches
Mail list logo