[ros-dev] [ros-diffs] [tkreuzer] 64757: [NTOSKRNL] Don't use an uninitialized variable in MmArmAccessFault (Alex, please review). Brought to you by MSVC runtime checks.

[Timo Kreuzer]

No. Not even coverity seems to have caught this.

Am 16.10.2014 07:59, schrieb Alex Ionescu:
> Wow, does static analyzer really not catch this?
>
> Best regards,
> Alex Ionescu
>
> On Wed, Oct 15, 2014 at 3:03 PM, <tkreuzer at svn.reactos.org 
> <mailto:tkreuzer at svn.reactos.org>> wrote:
>
>     Author: tkreuzer
>     Date: Wed Oct 15 22:03:50 2014
>     New Revision: 64757
>
>     URL: http://svn.reactos.org/svn/reactos?rev=64757&view=rev
>     Log:
>     [NTOSKRNL]
>     Don't use an uninitialized variable in MmArmAccessFault (Alex,
>     please review). Brought to you by MSVC runtime checks.
>
>     Modified:
>         trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c
>
>     Modified: trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c
>     URL:
>     http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c?rev=64757&r1=64756&r2=64757&view=diff
>     ==============================================================================
>     --- trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c   [iso-8859-1]
>     (original)
>     +++ trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c   [iso-8859-1] Wed
>     Oct 15 22:03:50 2014
>     @@ -1511,7 +1511,7 @@
>          NTSTATUS Status;
>          PMMSUPPORT WorkingSet;
>          ULONG ProtectionCode;
>     -    PMMVAD Vad;
>     +    PMMVAD Vad = NULL;
>          PFN_NUMBER PageFrameIndex;
>          ULONG Color;
>          BOOLEAN IsSessionAddress;
>
>
>
>
>
> _______________________________________________
> Ros-dev mailing list
> Ros-dev at reactos.org
> http://www.reactos.org/mailman/listinfo/ros-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.reactos.org/pipermail/ros-dev/attachments/20141016/74837d47/attachment.html>