Re: quartz: check hr so we are not incrementing a NULL (coverity 106)
6 Oct
2008
6 Oct
'08
7:41 a.m.
"Aric Stewart" <aric(a)codeweavers.com> wrote:
- wszPatternString++; /* skip ',' */ + if (hr == S_OK) + wszPatternString++; /* skip ',' */
if (hr == S_OK) {
Why not just move it into the below block? -- Dmitry.
6 Oct
6 Oct
7:42 a.m.
New subject: quartz: check hr so we are not incrementing a NULL (coverity 106)
true, that would be cleaner. I will resubmit. -aric Dmitry Timoshkov wrote:
"Aric Stewart" <aric(a)codeweavers.com> wrote:
- wszPatternString++; /* skip ',' */ + if (hr == S_OK) + wszPatternString++; /* skip ',' */
if (hr == S_OK) {
Why not just move it into the below block?
6278
Age (days ago)
6278
Last active (days ago)
1 comments
2 participants
participants (2)
-
Aric Stewart -
Dmitry Timoshkov