[Spice-devel] [PATCH 1/4] qxl-wddm-dod: Correct format of printouts

Yuri Benditovich yuri.benditovich at daynix.com
Mon Jul 3 15:21:34 UTC 2017


On Mon, Jul 3, 2017 at 11:58 AM, Christophe Fergeau <cfergeau at redhat.com>
wrote:

> On Fri, Jun 30, 2017 at 12:11:42PM +0300, Yuri Benditovich wrote:
> > Fixed wrong formats of printouts producing errors with WPP.
> > All fixed format strings were wrong mainly due to incorrect
> > format for pointers.
>
> Quickly glanced through this, looks good, anyway this would be caught at
> compile time if something was very wrong.
> Anything we can add to the project C/CXX flags so that it's caught early
> on?
>
>
Not yet, I did not find a quick way to force the compiler to do format
checking when building drivers.
The compiler should do it by default, but does not. We definitely do not
disable these warnings in the project and some of them are on W1.
And now as soon as we build the same source with WPP, they will be caught
anyway.


> Acked-by: Christophe Fergeau <cfergeau at redhat.com>
>
> Christophe
>
> >
> > Signed-off-by: Yuri Benditovich <yuri.benditovich at daynix.com>
> > ---
> >  qxldod/QxlDod.cpp | 102 +++++++++++++++++++++++++++---
> ------------------------
> >  qxldod/driver.cpp |   2 +-
> >  2 files changed, 52 insertions(+), 52 deletions(-)
> >
> > diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp
> > index 6f2ca88..b97301a 100755
> > --- a/qxldod/QxlDod.cpp
> > +++ b/qxldod/QxlDod.cpp
> > @@ -570,7 +570,7 @@ NTSTATUS QxlDod::Escape(_In_ CONST DXGKARG_ESCAPE*
> pEscape)
> >      NTSTATUS Status = STATUS_SUCCESS;
> >      QXL_ASSERT(pEscape != NULL);
> >
> > -    DbgPrint(TRACE_LEVEL_VERBOSE, ("<---> %s Flags = %d\n",
> __FUNCTION__, pEscape->Flags));
> > +    DbgPrint(TRACE_LEVEL_VERBOSE, ("<---> %s Flags = %d\n",
> __FUNCTION__, pEscape->Flags.Value));
> >
> >      Status = m_pHWDevice->Escape(pEscape);
> >      DbgPrint(TRACE_LEVEL_VERBOSE, ("<---> %s Status = %x\n",
> __FUNCTION__, Status));
> > @@ -714,7 +714,7 @@ NTSTATUS QxlDod::IsSupportedVidPn(_Inout_
> DXGKARG_ISSUPPORTEDVIDPN* pIsSupported
> >      NTSTATUS Status = m_DxgkInterface.DxgkCbQueryVid
> PnInterface(pIsSupportedVidPn->hDesiredVidPn,
> DXGK_VIDPN_INTERFACE_VERSION_V1, &pVidPnInterface);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("DxgkCbQueryVidPnInterface failed
> with Status = 0x%X, hDesiredVidPn = 0x%I64x\n", Status,
> pIsSupportedVidPn->hDesiredVidPn));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("DxgkCbQueryVidPnInterface failed
> with Status = 0x%X, hDesiredVidPn = 0x%p\n", Status,
> pIsSupportedVidPn->hDesiredVidPn));
> >          return Status;
> >      }
> >
> > @@ -723,7 +723,7 @@ NTSTATUS QxlDod::IsSupportedVidPn(_Inout_
> DXGKARG_ISSUPPORTEDVIDPN* pIsSupported
> >      Status = pVidPnInterface->pfnGetTopolog
> y(pIsSupportedVidPn->hDesiredVidPn, &hVidPnTopology,
> &pVidPnTopologyInterface);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetTopology failed with Status
> = 0x%X, hDesiredVidPn = 0x%I64x\n", Status, pIsSupportedVidPn->hDesiredVid
> Pn));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetTopology failed with Status
> = 0x%X, hDesiredVidPn = 0x%p\n", Status, pIsSupportedVidPn->hDesiredVid
> Pn));
> >          return Status;
> >      }
> >
> > @@ -738,7 +738,7 @@ NTSTATUS QxlDod::IsSupportedVidPn(_Inout_
> DXGKARG_ISSUPPORTEDVIDPN* pIsSupported
> >          }
> >          else if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetNumPathsFromSource
> failed with Status = 0x%X hVidPnTopology = 0x%I64x, SourceId = 0x%I64x",
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetNumPathsFromSource
> failed with Status = 0x%X hVidPnTopology = 0x%p, SourceId = 0x%x",
> >                             Status, hVidPnTopology, SourceId));
> >              return Status;
> >          }
> > @@ -803,7 +803,7 @@ NTSTATUS QxlDod::AddSingleSourceMode(_In_ CONST
> DXGK_VIDPNSOURCEMODESET_INTERFAC
> >          if (!NT_SUCCESS(Status))
> >          {
> >              // If failed to create a new mode info, mode doesn't need
> to be released since it was never created
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed
> with Status = 0x%X, hVidPnSourceModeSet = 0x%I64x", Status,
> hVidPnSourceModeSet));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed
> with Status = 0x%X, hVidPnSourceModeSet = 0x%p", Status,
> hVidPnSourceModeSet));
> >              return Status;
> >          }
> >
> > @@ -829,7 +829,7 @@ NTSTATUS QxlDod::AddSingleSourceMode(_In_ CONST
> DXGK_VIDPNSOURCEMODESET_INTERFAC
> >
> >              if (Status != STATUS_GRAPHICS_MODE_ALREADY_IN_MODESET)
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with
> Status = 0x%X, hVidPnSourceModeSet = 0x%I64x, pVidPnSourceModeInfo = %p",
> Status, hVidPnSourceModeSet, pVidPnSourceModeInfo));
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with
> Status = 0x%X, hVidPnSourceModeSet = 0x%p, pVidPnSourceModeInfo = %p",
> Status, hVidPnSourceModeSet, pVidPnSourceModeInfo));
> >                  return Status;
> >              }
> >          }
> > @@ -892,7 +892,7 @@ NTSTATUS QxlDod::AddSingleTargetMode(_In_ CONST
> DXGK_VIDPNTARGETMODESET_INTERFAC
> >          if (!NT_SUCCESS(Status))
> >          {
> >              // If failed to create a new mode info, mode doesn't need
> to be released since it was never created
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed
> with Status = 0x%X, hVidPnTargetModeSet = 0x%I64x", Status,
> hVidPnTargetModeSet));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed
> with Status = 0x%X, hVidPnTargetModeSet = 0x%p", Status,
> hVidPnTargetModeSet));
> >              return Status;
> >          }
> >          FillSignalInfo(pVidPnTargetModeInfo->VideoSignalInfo,
> pModeInfo, __FUNCTION__);
> > @@ -905,7 +905,7 @@ NTSTATUS QxlDod::AddSingleTargetMode(_In_ CONST
> DXGK_VIDPNTARGETMODESET_INTERFAC
> >          {
> >              if (Status != STATUS_GRAPHICS_MODE_ALREADY_IN_MODESET)
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with
> Status = 0x%X, hVidPnTargetModeSet = 0x%I64x, pVidPnTargetModeInfo = %p",
> Status, hVidPnTargetModeSet, pVidPnTargetModeInfo));
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with
> Status = 0x%X, hVidPnTargetModeSet = 0x%p, pVidPnTargetModeInfo = %p",
> Status, hVidPnTargetModeSet, pVidPnTargetModeInfo));
> >              }
> >
> >              // If adding the mode failed, release the mode, if this
> doesn't work there is nothing that can be done, some memory will get leaked
> > @@ -931,7 +931,7 @@ NTSTATUS QxlDod::AddSingleMonitorMode(_In_ CONST
> DXGKARG_RECOMMENDMONITORMODES*
> >      if (!NT_SUCCESS(Status))
> >      {
> >          // If failed to create a new mode info, mode doesn't need to be
> released since it was never created
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed with
> Status = 0x%X, hMonitorSourceModeSet = 0x%I64x", Status,
> pRecommendMonitorModes->hMonitorSourceModeSet));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed with
> Status = 0x%X, hMonitorSourceModeSet = 0x%p", Status,
> pRecommendMonitorModes->hMonitorSourceModeSet));
> >          return Status;
> >      }
> >
> > @@ -954,7 +954,7 @@ NTSTATUS QxlDod::AddSingleMonitorMode(_In_ CONST
> DXGKARG_RECOMMENDMONITORMODES*
> >      {
> >          if (Status != STATUS_GRAPHICS_MODE_ALREADY_IN_MODESET)
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with Status
> = 0x%X, hMonitorSourceModeSet = 0x%I64x, pMonitorSourceMode = 0x%I64x",
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with Status
> = 0x%X, hMonitorSourceModeSet = 0x%p, pMonitorSourceMode = 0x%p",
> >                              Status, pRecommendMonitorModes->hMonitorSourceModeSet,
> pMonitorSourceMode));
> >          }
> >          else
> > @@ -980,7 +980,7 @@ NTSTATUS QxlDod::AddSingleMonitorMode(_In_ CONST
> DXGKARG_RECOMMENDMONITORMODES*
> >          if (!NT_SUCCESS(Status))
> >          {
> >              // If failed to create a new mode info, mode doesn't need
> to be released since it was never created
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed
> with Status = 0x%X, hMonitorSourceModeSet = 0x%I64x", Status,
> pRecommendMonitorModes->hMonitorSourceModeSet));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnCreateNewModeInfo failed
> with Status = 0x%X, hMonitorSourceModeSet = 0x%p", Status,
> pRecommendMonitorModes->hMonitorSourceModeSet));
> >              return Status;
> >          }
> >
> > @@ -1004,7 +1004,7 @@ NTSTATUS QxlDod::AddSingleMonitorMode(_In_ CONST
> DXGKARG_RECOMMENDMONITORMODES*
> >          {
> >              if (Status != STATUS_GRAPHICS_MODE_ALREADY_IN_MODESET)
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with
> Status = 0x%X, hMonitorSourceModeSet = 0x%I64x, pMonitorSourceMode = 0x%p",
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAddMode failed with
> Status = 0x%X, hMonitorSourceModeSet = 0x%p, pMonitorSourceMode = 0x%p",
> >                                  Status, pRecommendMonitorModes->hMonitorSourceModeSet,
> pMonitorSourceMode));
> >              }
> >
> > @@ -1042,7 +1042,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >      NTSTATUS Status = m_DxgkInterface.DxgkCbQueryVid
> PnInterface(pEnumCofuncModality->hConstrainingVidPn,
> DXGK_VIDPN_INTERFACE_VERSION_V1, &pVidPnInterface);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("DxgkCbQueryVidPnInterface failed
> with Status = 0x%X, hFunctionalVidPn = 0x%I64x", Status,
> pEnumCofuncModality->hConstrainingVidPn));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("DxgkCbQueryVidPnInterface failed
> with Status = 0x%X, hFunctionalVidPn = 0x%p", Status,
> pEnumCofuncModality->hConstrainingVidPn));
> >          return Status;
> >      }
> >
> > @@ -1050,7 +1050,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >      Status = pVidPnInterface->pfnGetTopolog
> y(pEnumCofuncModality->hConstrainingVidPn, &hVidPnTopology,
> &pVidPnTopologyInterface);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetTopology failed with Status
> = 0x%X, hFunctionalVidPn = 0x%I64x", Status, pEnumCofuncModality->hConstrai
> ningVidPn));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetTopology failed with Status
> = 0x%X, hFunctionalVidPn = 0x%p", Status, pEnumCofuncModality->hConstrai
> ningVidPn));
> >          return Status;
> >      }
> >
> > @@ -1058,7 +1058,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >      Status = pVidPnTopologyInterface->pfnAc
> quireFirstPathInfo(hVidPnTopology, &pVidPnPresentPath);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireFirstPathInfo failed
> with Status =0x%X, hVidPnTopology = 0x%I64x", Status, hVidPnTopology));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireFirstPathInfo failed
> with Status =0x%X, hVidPnTopology = 0x%p", Status, hVidPnTopology));
> >          return Status;
> >      }
> >
> > @@ -1072,7 +1072,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >
> &pVidPnSourceModeSetInterface);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireSourceModeSet
> failed with Status = 0x%X, hConstrainingVidPn = 0x%I64x, SourceId =
> 0x%I64x",
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireSourceModeSet
> failed with Status = 0x%X, hConstrainingVidPn = 0x%p, SourceId = 0x%x",
> >                             Status, pEnumCofuncModality->hConstrainingVidPn,
> pVidPnPresentPath->VidPnSourceId));
> >              break;
> >          }
> > @@ -1081,7 +1081,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >          Status = pVidPnSourceModeSetInterface->
> pfnAcquirePinnedModeInfo(hVidPnSourceModeSet,
> &pVidPnPinnedSourceModeInfo);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePinnedModeInfo
> failed with Status = 0x%X, hVidPnSourceModeSet = 0x%I64x", Status,
> hVidPnSourceModeSet));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePinnedModeInfo
> failed with Status = 0x%X, hVidPnSourceModeSet = 0x%p", Status,
> hVidPnSourceModeSet));
> >              break;
> >          }
> >
> > @@ -1096,7 +1096,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >                  Status = pVidPnInterface->pfnReleaseSou
> rceModeSet(pEnumCofuncModality->hConstrainingVidPn, hVidPnSourceModeSet);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnReleaseSourceModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%I64x, hVidPnSourceModeSet = 0x%I64x",
> > +                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnReleaseSourceModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%p, hVidPnSourceModeSet = 0x%p",
> >                                     Status,
> pEnumCofuncModality->hConstrainingVidPn, hVidPnSourceModeSet));
> >                      break;
> >                  }
> > @@ -1109,7 +1109,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >
> &pVidPnSourceModeSetInterface);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnCreateNewSourceModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%I64x, SourceId = 0x%I64x",
> > +                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnCreateNewSourceModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%p, SourceId = 0x%x",
> >                                     Status,
> pEnumCofuncModality->hConstrainingVidPn, pVidPnPresentPath->VidPnSource
> Id));
> >                      break;
> >                  }
> > @@ -1121,7 +1121,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR, ("AddSingleSourceMode
> failed with Status = 0x%X, hFunctionalVidPn = 0x%I64x", Status,
> pEnumCofuncModality->hConstrainingVidPn));
> > +                    DbgPrint(TRACE_LEVEL_ERROR, ("AddSingleSourceMode
> failed with Status = 0x%X, hFunctionalVidPn = 0x%p", Status,
> pEnumCofuncModality->hConstrainingVidPn));
> >                      break;
> >                  }
> >
> > @@ -1129,7 +1129,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >                  Status = pVidPnInterface->pfnAssignSour
> ceModeSet(pEnumCofuncModality->hConstrainingVidPn,
> pVidPnPresentPath->VidPnSourceId, hVidPnSourceModeSet);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnAssignSourceModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%I64x, SourceId = 0x%I64x, hVidPnSourceModeSet = 0x%I64x",
> > +                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnAssignSourceModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%p, SourceId = 0x%x, hVidPnSourceModeSet = 0x%p",
> >                                     Status,
> pEnumCofuncModality->hConstrainingVidPn, pVidPnPresentPath->VidPnSourceId,
> hVidPnSourceModeSet));
> >                      break;
> >                  }
> > @@ -1148,7 +1148,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >
> &pVidPnTargetModeSetInterface);
> >              if (!NT_SUCCESS(Status))
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireTargetModeSet
> failed with Status = 0x%X, hConstrainingVidPn = 0x%I64x, TargetId =
> 0x%I64x",
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireTargetModeSet
> failed with Status = 0x%X, hConstrainingVidPn = 0x%p, TargetId = 0x%x",
> >                                 Status, pEnumCofuncModality->hConstrainingVidPn,
> pVidPnPresentPath->VidPnTargetId));
> >                  break;
> >              }
> > @@ -1156,7 +1156,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >              Status = pVidPnTargetModeSetInterface->
> pfnAcquirePinnedModeInfo(hVidPnTargetModeSet,
> &pVidPnPinnedTargetModeInfo);
> >              if (!NT_SUCCESS(Status))
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePinnedModeInfo
> failed with Status = 0x%X, hVidPnTargetModeSet = 0x%I64x", Status,
> hVidPnTargetModeSet));
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePinnedModeInfo
> failed with Status = 0x%X, hVidPnTargetModeSet = 0x%p", Status,
> hVidPnTargetModeSet));
> >                  break;
> >              }
> >
> > @@ -1167,7 +1167,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >                  Status = pVidPnInterface->pfnReleaseTar
> getModeSet(pEnumCofuncModality->hConstrainingVidPn, hVidPnTargetModeSet);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnReleaseTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%I64x, hVidPnTargetModeSet = 0x%I64x",
> > +                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnReleaseTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%p, hVidPnTargetModeSet = 0x%p",
> >                                         Status,
> pEnumCofuncModality->hConstrainingVidPn, hVidPnTargetModeSet));
> >                      break;
> >                  }
> > @@ -1180,7 +1180,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >
> &pVidPnTargetModeSetInterface);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnCreateNewTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%I64x, TargetId = 0x%I64x",
> > +                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnCreateNewTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%p, TargetId = 0x%x",
> >                                     Status,
> pEnumCofuncModality->hConstrainingVidPn, pVidPnPresentPath->VidPnTarget
> Id));
> >                      break;
> >                  }
> > @@ -1189,7 +1189,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR, ("AddSingleTargetMode
> failed with Status = 0x%X, hFunctionalVidPn = 0x%I64x", Status,
> pEnumCofuncModality->hConstrainingVidPn));
> > +                    DbgPrint(TRACE_LEVEL_ERROR, ("AddSingleTargetMode
> failed with Status = 0x%X, hFunctionalVidPn = 0x%p", Status,
> pEnumCofuncModality->hConstrainingVidPn));
> >                      break;
> >                  }
> >
> > @@ -1197,7 +1197,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >                  Status = pVidPnInterface->pfnAssignTarg
> etModeSet(pEnumCofuncModality->hConstrainingVidPn,
> pVidPnPresentPath->VidPnTargetId, hVidPnTargetModeSet);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnAssignTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%I64x, TargetId = 0x%I64x, hVidPnTargetModeSet = 0x%I64x",
> > +                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnAssignTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%p, TargetId = 0x%x, hVidPnTargetModeSet = 0x%p",
> >                                     Status,
> pEnumCofuncModality->hConstrainingVidPn, pVidPnPresentPath->VidPnTargetId,
> hVidPnTargetModeSet));
> >                      break;
> >                  }
> > @@ -1209,7 +1209,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >                  Status = pVidPnTargetModeSetInterface->
> pfnReleaseModeInfo(hVidPnTargetModeSet, pVidPnPinnedTargetModeInfo);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleaseModeInfo
> failed with Status = 0x%X, hVidPnTargetModeSet = 0x%I64x,
> pVidPnPinnedTargetModeInfo = %p",
> > +                    DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleaseModeInfo
> failed with Status = 0x%X, hVidPnTargetModeSet = 0x%p,
> pVidPnPinnedTargetModeInfo = %p",
> >                                          Status, hVidPnTargetModeSet,
> pVidPnPinnedTargetModeInfo));
> >                      break;
> >                  }
> > @@ -1219,7 +1219,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >                  Status = pVidPnInterface->pfnReleaseTar
> getModeSet(pEnumCofuncModality->hConstrainingVidPn, hVidPnTargetModeSet);
> >                  if (!NT_SUCCESS(Status))
> >                  {
> > -                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnReleaseTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%I64x, hVidPnTargetModeSet = 0x%I64x",
> > +                    DbgPrint(TRACE_LEVEL_ERROR,
> ("pfnReleaseTargetModeSet failed with Status = 0x%X, hConstrainingVidPn =
> 0x%p, hVidPnTargetModeSet = 0x%p",
> >                                         Status,
> pEnumCofuncModality->hConstrainingVidPn, hVidPnTargetModeSet));
> >                      break;
> >                  }
> > @@ -1233,7 +1233,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >              Status = pVidPnSourceModeSetInterface->
> pfnReleaseModeInfo(hVidPnSourceModeSet, pVidPnPinnedSourceModeInfo);
> >              if (!NT_SUCCESS(Status))
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleaseModeInfo failed
> with Status = 0x%X, hVidPnSourceModeSet = 0x%I64x,
> pVidPnPinnedSourceModeInfo = %p",
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleaseModeInfo failed
> with Status = 0x%X, hVidPnSourceModeSet = 0x%p, pVidPnPinnedSourceModeInfo
> = %p",
> >                                      Status, hVidPnSourceModeSet,
> pVidPnPinnedSourceModeInfo));
> >                  break;
> >              }
> > @@ -1246,7 +1246,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >              Status = pVidPnInterface->pfnReleaseSou
> rceModeSet(pEnumCofuncModality->hConstrainingVidPn, hVidPnSourceModeSet);
> >              if (!NT_SUCCESS(Status))
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleaseSourceModeSet
> failed with Status = 0x%X, hConstrainingVidPn = 0x%I64x,
> hVidPnSourceModeSet = 0x%I64x",
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleaseSourceModeSet
> failed with Status = 0x%X, hConstrainingVidPn = 0x%p, hVidPnSourceModeSet =
> 0x%p",
> >                                 Status, pEnumCofuncModality->hConstrainingVidPn,
> hVidPnSourceModeSet));
> >                  break;
> >              }
> > @@ -1296,7 +1296,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >              Status = pVidPnTopologyInterface->pfnUp
> datePathSupportInfo(hVidPnTopology, &LocalVidPnPresentPath);
> >              if (!NT_SUCCESS(Status))
> >              {
> > -                DbgPrint(TRACE_LEVEL_ERROR, ("pfnUpdatePathSupportInfo
> failed with Status = 0x%X, hVidPnTopology = 0x%I64x", Status,
> hVidPnTopology));
> > +                DbgPrint(TRACE_LEVEL_ERROR, ("pfnUpdatePathSupportInfo
> failed with Status = 0x%X, hVidPnTopology = 0x%p", Status, hVidPnTopology));
> >                  break;
> >              }
> >          }
> > @@ -1307,7 +1307,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >          Status = pVidPnTopologyInterface->pfnAc
> quireNextPathInfo(hVidPnTopology, pVidPnPresentPathTemp,
> &pVidPnPresentPath);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireNextPathInfo failed
> with Status = 0x%X, hVidPnTopology = 0x%I64x, pVidPnPresentPathTemp = %p",
> Status, hVidPnTopology, pVidPnPresentPathTemp));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireNextPathInfo failed
> with Status = 0x%X, hVidPnTopology = 0x%p, pVidPnPresentPathTemp = %p",
> Status, hVidPnTopology, pVidPnPresentPathTemp));
> >              break;
> >          }
> >
> > @@ -1315,7 +1315,7 @@ NTSTATUS QxlDod::EnumVidPnCofuncModality(_In_
> CONST DXGKARG_ENUMVIDPNCOFUNCMODAL
> >          NTSTATUS TempStatus = pVidPnTopologyInterface->pfnReleasePathInfo(hVidPnTopology,
> pVidPnPresentPathTemp);
> >          if (!NT_SUCCESS(TempStatus))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleasePathInfo failed
> with Status = 0x%X, hVidPnTopology = 0x%I64x, pVidPnPresentPathTemp = %p",
> TempStatus, hVidPnTopology, pVidPnPresentPathTemp));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleasePathInfo failed
> with Status = 0x%X, hVidPnTopology = 0x%p, pVidPnPresentPathTemp = %p",
> TempStatus, hVidPnTopology, pVidPnPresentPathTemp));
> >              Status = TempStatus;
> >              break;
> >          }
> > @@ -1438,7 +1438,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >      Status = m_DxgkInterface.DxgkCbQueryVid
> PnInterface(pCommitVidPn->hFunctionalVidPn, DXGK_VIDPN_INTERFACE_VERSION_V1,
> &pVidPnInterface);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("DxgkCbQueryVidPnInterface failed
> with Status = 0x%X, hFunctionalVidPn = 0x%I64x", Status,
> pCommitVidPn->hFunctionalVidPn));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("DxgkCbQueryVidPnInterface failed
> with Status = 0x%X, hFunctionalVidPn = 0x%p", Status,
> pCommitVidPn->hFunctionalVidPn));
> >          goto CommitVidPnExit;
> >      }
> >
> > @@ -1446,7 +1446,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >      Status = pVidPnInterface->pfnGetTopolog
> y(pCommitVidPn->hFunctionalVidPn, &hVidPnTopology,
> &pVidPnTopologyInterface);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetTopology failed with Status
> = 0x%X, hFunctionalVidPn = 0x%I64x", Status, pCommitVidPn->hFunctionalVidPn
> ));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetTopology failed with Status
> = 0x%X, hFunctionalVidPn = 0x%p", Status, pCommitVidPn->hFunctionalVidPn
> ));
> >          goto CommitVidPnExit;
> >      }
> >
> > @@ -1454,7 +1454,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >      Status = pVidPnTopologyInterface->pfnGetNumPaths(hVidPnTopology,
> &NumPaths);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetNumPaths failed with Status
> = 0x%X, hVidPnTopology = 0x%I64x", Status, hVidPnTopology));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetNumPaths failed with Status
> = 0x%X, hVidPnTopology = 0x%p", Status, hVidPnTopology));
> >          goto CommitVidPnExit;
> >      }
> >
> > @@ -1467,7 +1467,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >
> &pVidPnSourceModeSetInterface);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireSourceModeSet
> failed with Status = 0x%X, hFunctionalVidPn = 0x%I64x, SourceId = 0x%I64x",
> Status, pCommitVidPn->hFunctionalVidPn, pCommitVidPn->AffectedVidPnSou
> rceId));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquireSourceModeSet
> failed with Status = 0x%X, hFunctionalVidPn = 0x%p, SourceId = 0x%x",
> Status, pCommitVidPn->hFunctionalVidPn, pCommitVidPn->AffectedVidPnSou
> rceId));
> >              goto CommitVidPnExit;
> >          }
> >
> > @@ -1475,7 +1475,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >          Status = pVidPnSourceModeSetInterface->
> pfnAcquirePinnedModeInfo(hVidPnSourceModeSet,
> &pPinnedVidPnSourceModeInfo);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePinnedModeInfo
> failed with Status = 0x%X, hFunctionalVidPn = 0x%I64x", Status,
> pCommitVidPn->hFunctionalVidPn));
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePinnedModeInfo
> failed with Status = 0x%X, hFunctionalVidPn = 0x%p", Status,
> pCommitVidPn->hFunctionalVidPn));
> >              goto CommitVidPnExit;
> >          }
> >      }
> > @@ -1513,7 +1513,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >      Status = pVidPnTopologyInterface->pfnGe
> tNumPathsFromSource(hVidPnTopology, pCommitVidPn->AffectedVidPnSourceId,
> &NumPathsFromSource);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetNumPathsFromSource failed
> with Status = 0x%X, hVidPnTopology = 0x%I64x", Status, hVidPnTopology));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("pfnGetNumPathsFromSource failed
> with Status = 0x%X, hVidPnTopology = 0x%p", Status, hVidPnTopology));
> >          goto CommitVidPnExit;
> >      }
> >
> > @@ -1525,7 +1525,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >          Status = pVidPnTopologyInterface->pfnEn
> umPathTargetsFromSource(hVidPnTopology, pCommitVidPn->AffectedVidPnSourceId,
> PathIndex, &TargetId);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnEnumPathTargetsFromSource
> failed with Status = 0x%X, hVidPnTopology = 0x%I64x, SourceId = 0x%I64x,
> PathIndex = 0x%I64x",
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnEnumPathTargetsFromSource
> failed with Status = 0x%X, hVidPnTopology = 0x%p, SourceId = 0x%x,
> PathIndex = 0x%I64x",
> >                              Status, hVidPnTopology,
> pCommitVidPn->AffectedVidPnSourceId, PathIndex));
> >              goto CommitVidPnExit;
> >          }
> > @@ -1534,7 +1534,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >          Status = pVidPnTopologyInterface->pfnAcquirePathInfo(hVidPnTopology,
> pCommitVidPn->AffectedVidPnSourceId, TargetId, &pVidPnPresentPath);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePathInfo failed
> with Status = 0x%X, hVidPnTopology = 0x%I64x, SourceId = 0x%I64x, TargetId
> = 0x%I64x",
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnAcquirePathInfo failed
> with Status = 0x%X, hVidPnTopology = 0x%p, SourceId = 0x%x, TargetId =
> 0x%x",
> >                              Status, hVidPnTopology,
> pCommitVidPn->AffectedVidPnSourceId, TargetId));
> >              goto CommitVidPnExit;
> >          }
> > @@ -1554,7 +1554,7 @@ NTSTATUS QxlDod::CommitVidPn(_In_ CONST
> DXGKARG_COMMITVIDPN* CONST pCommitVidPn)
> >          Status = pVidPnTopologyInterface->pfnReleasePathInfo(hVidPnTopology,
> pVidPnPresentPath);
> >          if (!NT_SUCCESS(Status))
> >          {
> > -            DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleasePathInfo failed
> with Status = 0x%X, hVidPnTopoogy = 0x%I64x, pVidPnPresentPath = %p",
> > +            DbgPrint(TRACE_LEVEL_ERROR, ("pfnReleasePathInfo failed
> with Status = 0x%X, hVidPnTopoogy = 0x%p, pVidPnPresentPath = %p",
> >                              Status, hVidPnTopology, pVidPnPresentPath));
> >              goto CommitVidPnExit;
> >          }
> > @@ -1933,7 +1933,7 @@ NTSTATUS QxlDod::RegisterHWInfo(_In_ ULONG Id)
> >      Status = IoOpenDeviceRegistryKey(m_pPhysicalDevice,
> PLUGPLAY_REGKEY_DRIVER, KEY_SET_VALUE, &DevInstRegKeyHandle);
> >      if (!NT_SUCCESS(Status))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("IoOpenDeviceRegistryKey failed
> for PDO: 0x%I64x, Status: 0x%X", m_pPhysicalDevice, Status));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("IoOpenDeviceRegistryKey failed
> for PDO: 0x%p, Status: 0x%X", m_pPhysicalDevice, Status));
> >          return Status;
> >      }
> >
> > @@ -2360,7 +2360,7 @@ VOID BltBits (
> >      #pragma prefast(suppress: __WARNING_EXCEPTIONEXECUTEHANDLER,
> "try/except is only able to protect against user-mode errors and these are
> the only errors we try to catch here");
> >      __except(EXCEPTION_EXECUTE_HANDLER)
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("Either dst (0x%I64x) or src
> (0x%I64x) bits encountered exception during access.\n", pDst->pBits,
> pSrc->pBits));
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("Either dst (0x%p) or src (0x%p)
> bits encountered exception during access.\n", pDst->pBits, pSrc->pBits));
> >      }
> >  }
> >
> > @@ -3377,7 +3377,7 @@ NTSTATUS QxlDevice::HWInit(PCM_RESOURCE_LIST
> pResList, DXGK_DISPLAY_INFORMATION*
> >                         m_IoBase = (PUCHAR)(ULONG_PTR)pResDescrip
> tor->u.Port.Start.QuadPart;
> >                         m_IoSize = pResDescriptor->u.Port.Length;
> >                     }
> > -                   DbgPrint(TRACE_LEVEL_VERBOSE, ("io_base  [%X-%X]\n",
> > +                   DbgPrint(TRACE_LEVEL_VERBOSE, ("io_base  [%p-%p]\n",
> >                                   m_IoBase,
> >                                   m_IoBase +
> >                                   m_IoSize));
> > @@ -4084,7 +4084,7 @@ UINT64 QxlDevice::ReleaseOutput(UINT64 output_id)
> >      UINT64 next;
> >
> >      ASSERT(output_id);
> > -    DbgPrint(TRACE_LEVEL_VERBOSE, ("--->%s 0x%x\n", __FUNCTION__,
> output));
> > +    DbgPrint(TRACE_LEVEL_VERBOSE, ("--->%s 0x%p\n", __FUNCTION__,
> output));
> >
> >      for (now = output->resources, end = now + output->num_res; now <
> end; now++) {
> >          RELEASE_RES(*now);
> > @@ -4157,7 +4157,7 @@ void *QxlDevice::AllocMem(UINT32 mspace_type,
> size_t size, BOOL force)
> >
> >      ASSERT((!ptr && (!force || !m_bActive)) || (ptr >=
> m_MSInfo[mspace_type].mspace_start &&
> >                                        ptr <
> m_MSInfo[mspace_type].mspace_end));
> > -    DbgPrint(TRACE_LEVEL_VERBOSE, ("<---%s: ptr 0x%x\n", __FUNCTION__,
> ptr));
> > +    DbgPrint(TRACE_LEVEL_VERBOSE, ("<---%s: ptr 0x%p\n", __FUNCTION__,
> ptr));
> >      return ptr;
> >  }
> >
> > @@ -4198,7 +4198,7 @@ QXLDrawable *QxlDevice::GetDrawable()
> >      output->num_res = 0;
> >      RESOURCE_TYPE(output, RESOURCE_TYPE_DRAWABLE);
> >      ((QXLDrawable *)output->data)->release_info.id = (UINT64)output;
> > -    DbgPrint(TRACE_LEVEL_VERBOSE, ("<--> %s 0x%x\n", __FUNCTION__,
> output));
> > +    DbgPrint(TRACE_LEVEL_VERBOSE, ("<--> %s 0x%p\n", __FUNCTION__,
> output));
> >      return(QXLDrawable *)output->data;
> >  }
> >
> > @@ -4627,7 +4627,7 @@ QXLDrawable *QxlDevice::PrepareBltBits (
> >          DiscardDrawable(drawable);
> >          drawable = NULL;
> >      } else {
> > -        DbgPrint(TRACE_LEVEL_INFORMATION, ("%s drawable= %p type = %d,
> effect = %d Dest right(%d) left(%d) top(%d) bottom(%d) src_bitmap= %p.\n",
> __FUNCTION__,
> > +        DbgPrint(TRACE_LEVEL_INFORMATION, ("%s drawable= %p type = %d,
> effect = %d Dest right(%d) left(%d) top(%d) bottom(%d) src_bitmap=
> %I64x.\n", __FUNCTION__,
> >              drawable, drawable->type, drawable->effect,
> drawable->surfaces_rects[0].right, drawable->surfaces_rects[0].left,
> >              drawable->surfaces_rects[0].top,
> drawable->surfaces_rects[0].bottom,
> >              drawable->u.copy.src_bitmap));
> > @@ -4881,7 +4881,7 @@ NTSTATUS QxlDevice::SetCustomDisplay(QXLEscapeSetCustomDisplay*
> custom_display)
> >      UINT bpp = QXL_BPP;
> >      DbgPrint(TRACE_LEVEL_WARNING, ("%s - %d (%dx%d#%d)\n",
> __FUNCTION__, m_Id, xres, yres, bpp));
> >      if (xres < MIN_WIDTH_SIZE || yres < MIN_HEIGHT_SIZE) {
> > -        DbgPrint(TRACE_LEVEL_VERBOSE, ("%s: (%dx%d#%d) less than
> (%dxd)\n", __FUNCTION__,
> > +        DbgPrint(TRACE_LEVEL_VERBOSE, ("%s: (%dx%d#%d) less than
> (%dx%d)\n", __FUNCTION__,
> >              xres, yres, bpp, MIN_WIDTH_SIZE, MIN_HEIGHT_SIZE));
> >      }
> >      m_CustomMode =(USHORT) ((m_CustomMode == m_ModeCount-1)?
> m_ModeCount - 2 : m_ModeCount - 1);
> > diff --git a/qxldod/driver.cpp b/qxldod/driver.cpp
> > index 2736eb4..1ff2abb 100755
> > --- a/qxldod/driver.cpp
> > +++ b/qxldod/driver.cpp
> > @@ -170,7 +170,7 @@ DodAddDevice(
> >      if ((pPhysicalDeviceObject == NULL) ||
> >          (ppDeviceContext == NULL))
> >      {
> > -        DbgPrint(TRACE_LEVEL_ERROR, ("One of pPhysicalDeviceObject
> (0x%I64x), ppDeviceContext (0x%I64x) is NULL",
> > +        DbgPrint(TRACE_LEVEL_ERROR, ("One of pPhysicalDeviceObject
> (0x%p), ppDeviceContext (0x%p) is NULL",
> >                          pPhysicalDeviceObject, ppDeviceContext));
> >          return STATUS_INVALID_PARAMETER;
> >      }
> > --
> > 2.7.0.windows.1
> >
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170703/14ad93a5/attachment-0001.html>


More information about the Spice-devel mailing list