Home
Examples of errors detected by the V519 diagnostic

Examples of errors detected by the V519 diagnostic

V519. The 'x' variable is assigned values twice successively. Perhaps this is a mistake.

Crystal Space 3D SDK

V519 The 'sampleDistanceReal' object is assigned values twice successively. Perhaps this is a mistake. plgsimpleformer simpleformer.cpp 642


csVector3 sampleDistanceReal;

void csSimpleSampler::CachePositions ()
{
  ...
  // Compute distance between sample points
  sampleDistanceReal = (resx == 1) ? 0 :
    (maxCorner.x-minCorner.x)/(float)(resx-1);
  sampleDistanceReal = (resz == 1) ? 0 :
    (maxCorner.z-minCorner.z)/(float)(resz-1);
  ...
}

Most likely this is what should be written here: sampleDistanceReal.x = ...; sampleDistanceReal.z = ...;

Ultimate TCP/IP

V519 The 'prev' object is assigned values twice successively. Perhaps this is a mistake. UTHistory uh_ctrl.cpp 787


int CUH_Control::SetHistoryLength(int len){
  ...
  prev = m_HLEndPosPtr->m_prev;
  prev = m_HLEndPosPtr->m_next = NULL;
  ...
}

XUIFramework

V519 The 'm_xSt' object is assigned values twice successively. Perhaps this is a mistake. Borne resizedlg.cpp 244


CResizeDlg::CResizeDlg(....)
  : CDialog(resID,pParent)
{
  m_xSt = CST_RESIZE;
  m_xSt = CST_RESIZE;
  m_xMin = 32;
  m_yMin = 32;
  m_nDelaySide = 0;
}

eMule

V519 The 'm_clrSample' object is assigned values twice successively. Perhaps this is a mistake. emule fontpreviewcombo.cpp 61


CFontPreviewCombo::CFontPreviewCombo()
{
  ...
  m_clrSample = GetSysColor(COLOR_WINDOWTEXT);
  m_clrSample = RGB(60,0,0);
  ...
}

Lugaru

V519 The 'radius' object is assigned values twice successively. Perhaps this is a mistake. Lugaru gamedraw.cpp 1505


int Game::DrawGLScene(void)
{
  ...
  radius=fast_sqrt(maxdistance);
  radius=110;
  ...
}

Wolfenstein 3D

V519 The 'r_shadows' object is assigned values twice successively. Perhaps this is a mistake. renderer tr_init.c 1135


void R_Register( void ) {
  ...
  r_shadows = ri.Cvar_Get( "cg_shadows", "1", 0 );
  r_shadows = ri.Cvar_Get( "cg_shadows", "1", 0 );
  ...
}

IPP Samples

V519 The 'MaxBrefPOC' object is assigned values twice successively. Perhaps this is a mistake. h264_enc umc_h264_enc_cpb_tmpl.cpp.h 784


H264EncoderFrameType*
H264ENC_MAKE_NAME(H264EncoderFrameList_findOldestToEncode)(....)
{
  ...
  MaxBrefPOC =
   H264ENC_MAKE_NAME(H264EncoderFrame_PicOrderCnt)(pCurr, 0, 3);
  MaxBrefPOC =
   H264ENC_MAKE_NAME(H264EncoderFrame_PicOrderCnt)(pCurr, 0, 3);
  ...
}

IPP Samples

V519 The 'k' object is assigned values twice successively. Perhaps this is a mistake. aac_enc sbr_enc_resampler_fp.c 90


AACStatus sbrencResampler_v2_32f(Ipp32f* pSrc, Ipp32f* pDst)
{
  ...
  k = nCoef-1;
  k = nCoef;
  ...
}

IPP Samples

V519 The '* tmpPtr' object is assigned values twice successively. Perhaps this is a mistake. aac_dec als_dec_api.c 928


AACStatus alsdecGetFrame(....)
{
  ...
  for (i = 0; i < num; i++) {
    ...
    *tmpPtr = (Ipp32s)((tmp << 24) + ((tmp & 0xff00) << 8) +
                      ((tmp >> 8) & 0xff00) + (tmp >> 24));
    *tmpPtr = *srcPrt;
    ...
  }
  ...
}

IPP Samples

V519 The 'saveXMask' object is assigned values twice successively. Perhaps this is a mistake. ipl iplremap.c 36

V519 The 'saveYMask' object is assigned values twice successively. Perhaps this is a mistake. ipl iplremap.c 38


static
IPLStatus ownRemap8u_Pixel(....) {
  ...
  saveXMask    = xMap->maskROI;

  saveYMask    = yMap->maskROI;
  saveYMask    = NULL;
  ...
}

Identical errors can be found in some other places:

V519 The 'saveXMask' object is assigned values twice successively. Perhaps this is a mistake. ipl iplremap.c 123
V519 The 'saveYMask' object is assigned values twice successively. Perhaps this is a mistake. ipl iplremap.c 125
V519 The 'saveXMask' object is assigned values twice successively. Perhaps this is a mistake. ipl iplremap.c 206
And 3 additional diagnostic messages.

IPP Samples

V519 The 'mNumOfFrames' object is assigned values twice successively. Perhaps this is a mistake. mpeg4_enc mp4_enc_misc.cpp 276


Ipp32s ippVideoEncoderMPEG4::Init(mp4_Param *par)
{
  ...
  mNumOfFrames = par->NumOfFrames;
  mNumOfFrames = -1;
  ...
}

IT++

V519 The 'flag' object is assigned values twice successively. Perhaps this is a mistake. itpp_mkl vq.cpp 170


ifstream &operator>>(ifstream &ifs, vec &v)
{
  ...
  flag = true;
  flag = false;
  ...
}

Strange code. The 'flag' variable is not used further at all.

Newton Game Dynamics

V519 The 'damp' object is assigned values twice successively. Perhaps this is a mistake. physics dgbody.cpp 404


void dgBody::AddBuoyancyForce (....)
{
  ...
  damp = (m_omega % m_omega) * dgFloat32 (10.0f) *
         fluidAngularViscousity;
  damp = GetMax (GetMin ((m_omega % m_omega) *
                         dgFloat32 (1000.0f) *
                         fluidAngularViscousity,
                         dgFloat32(0.25f)),
                 dgFloat32(2.0f));
  ...
}

QT

V519 The 'tos' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 811, 819. bootstrap qxmlstream.cpp 819


void QXmlStreamReaderPrivate::init()
{
  tos = 0;                   <<<---
  scanDtd = false;
  token = -1;
  token_char = 0;
  isEmptyElement = false;
  isWhitespace = true;
  isCDATA = false;
  standalone = false;
  tos = 0;                   <<<---
  ...
}

QT

V519 The 'ret' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 3765, 3767. QtGui qcleanlooksstyle.cpp 3767


int QCleanlooksStyle::pixelMetric(....)
{
  ... //cases

  case PM_SpinBoxFrameWidth:
    ret = 3;
    break;
  case PM_MenuBarItemSpacing:
    ret = 6;
  case PM_MenuBarHMargin:
    ret = 0;
    break;

  ... //cases
}

The break operator is missing.

QT

V519 The 'x1' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 2218, 2219. Qt3Support q3canvas.cpp 2219


bool qt_testCollision(const Q3CanvasSprite* s1,
                      const Q3CanvasSprite* s2)
{
  ...
  t=x1; x1=x2; x2=t;
  t=y1; x1=y2; y2=t;
  ...
}

This is what should have been written here: t=y1; y1=y2; y2=t;

QT

V519 The 'selectable' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 646, 652. Qt3Support q3listview.cpp 652


void Q3ListViewItem::init()
{
  ...
  selectable = true;        <<<---
  lsc = Unsorted;
  lso = true;
  configured = false;
  expandable = false;
  selectable = true;        <<<---
  ...
}

QT

V519 The 'retVal' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 169, 171. QtWebKit inspectorclientqt.cpp 171


static String variantToSetting(const QVariant& qvariant)
{
    String retVal;

    switch (qvariant.type()) {
    case QVariant::Bool:
        retVal = qvariant.toBool() ? "true" : "false";
    case QVariant::String:
        retVal = qvariant.toString();
    default:
        break;
    }

    return retVal;
}

The break operator is missing.

QT

V519 The 'shortVersion' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1791, 1795. qdoc3 htmlgenerator.cpp 1795


void HtmlGenerator::generateHeader(....)
{
  ...
  QString shortVersion;
  shortVersion = project + " " + shortVersion + ": ";
  if (node && !node->doc().location().isEmpty())
      out() << "<!-- " << node->doc().location().fileName() <<
      " -->\n";

  shortVersion = myTree->version();
  ...
}

Most likely this is what should be written here: shortVersion += myTree->version();

QT

V545 Such conditional expression of 'if' operator is incorrect for the HRESULT type value '(HRESULT) 0L'. The SUCCEEDED or FAILED macro should be used instead. phonon_ds9 qbasefilter.cpp 60


STDMETHODIMP QEnumPins::QueryInterface(const IID &iid,void **out)
{
  ...
  if (S_OK)
    AddRef();
  return hr;
}

It works on a sheer luck. S_OK is 0, by the way. That is, we don't perform AddRef(). This is what should have been written here: if (hr == S_OK).

MySQL

V519 The 'stats.max_data_file_length' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1597, 1598. archive ha_archive.cc 1598


int ha_archive::info(uint flag)
{
  ...
  stats.max_data_file_length= share->rows_recorded *
                              stats.mean_rec_length;
  stats.max_data_file_length= MAX_FILE_SIZE;
  ...
}

CMake

V519 The 'p [1]' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 272, 273. cmlibarchive archive_write_set_format_pax.c 273


static char *
utf8_encode(const wchar_t *wval)
{
  ...
  p[0] = 0xfc | ((wc >> 30) & 0x01);
  p[1] = 0x80 | ((wc >> 24) & 0x3f); <<<---
  p[1] = 0x80 | ((wc >> 18) & 0x3f); <<<---
  p[2] = 0x80 | ((wc >> 12) & 0x3f);

Fireflies

V519 The 'm_pDialog' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 3470, 3481. DXUTOpt dxutgui.cpp 3481


CDXUTControl::CDXUTControl( CDXUTDialog* pDialog )
{
  ...
  m_pDialog = pDialog;
  ...
  m_pDialog = NULL;
  ...
}

Intel AMT SDK

V519 The '* state' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1049, 1050. StorageHelper storagehelper.c 1050


PT_STATUS OpenPTSession(....)
{
  ...
  *state = PT_REGISTERED;
  *state = PT_OPENED;
  ...
}

Clang

V519 The 'SignedOverflowBehavior' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 207, 213. FrontendTests langoptions.h 213


LangOptions() {
  ...
  SignedOverflowBehavior = SOB_Undefined;

  AssumeSaneOperatorNew = 1;
  AccessControl = 1;
  ElideConstructors = 1;

  SignedOverflowBehavior = 0;
  ...
}

Clang

V519 The 'Src1Name' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 211, 215. LLVMX86AsmPrinter x86instcomments.cpp 215


void llvm::EmitAnyX86InstComments(....) {
  ...
  case X86::VPERMILPSri:
    DecodeVPERMILPSMask(4, MI->getOperand(2).getImm(),
                        ShuffleMask);
    Src1Name = getRegName(MI->getOperand(0).getReg());
  case X86::VPERMILPSYri:
    DecodeVPERMILPSMask(8, MI->getOperand(2).getImm(),
                        ShuffleMask);
    Src1Name = getRegName(MI->getOperand(0).getReg());
    break;
  ...
}

The break operator is missing.

Clang

V519 The 'ParmOffset' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 3953, 3956. clangAST astcontext.cpp 3956


std::string ASTContext::getObjCEncodingForBlock(....) const {
  ...
  ParmOffset = PtrSize;
  // Argument types.
  ParmOffset = PtrSize;
  ...
}

ReactOS

V519 The 'bih.bV5AlphaMask' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 357, 358. windowscodecs bmpencode.c 358


static HRESULT WINAPI BmpFrameEncode_Commit(....)
{
  ...
  bih.bV5RedMask = This->format->redmask;
  bih.bV5GreenMask = This->format->greenmask;
  bih.bV5BlueMask = This->format->bluemask;
  bih.bV5AlphaMask = This->format->alphamask;  <<<---
  bih.bV5AlphaMask = LCS_DEVICE_RGB;           <<<---
  ...
}

Most likely this is what should be written here: bih.bV5CSType = LCS_DEVICE_RGB;

ReactOS

V519 The 'v2' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 343, 343. win32k gradient.c 343


#define SWAP(a,b,c)  c = a;\
                     a = b;\
                     a = c

BOOL FASTCALL
IntEngGradientFillTriangle(....)
{
  ...
  SWAP(v2,v3,t);
  ...
}

Masterpiece! This is what should have been written here: c = a; a = b; b = c;

ReactOS

V519 The 'CenterCount' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 4125, 4126. win32k dib32gen.c 4126


static void
DIB_32BPP_BitBlt_SRCCOPY(PBLTINFO BltInfo)
{
  ...
  CenterCount = BltInfo->DestRect.right -
                BltInfo->DestRect.left;
  CenterCount = 4 * (BltInfo->DestRect.right -
                      BltInfo->DestRect.left);
  ...
}

Identical errors can be found in some other places:

V519 The 'CenterCount' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 4102, 4103. win32k dib32gen.c 4103

ReactOS

V519 The 'DriverObject->MajorFunction [0x0f]' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1058, 1059. scsiport scsiport.c 1059


#define IRP_MJ_INTERNAL_DEVICE_CONTROL    0x0f
#define IRP_MJ_SCSI                       0x0f

ULONG NTAPI
ScsiPortInitialize(....)
{
  ...
  DriverObject->MajorFunction[IRP_MJ_INTERNAL_DEVICE_CONTROL] =
    ScsiPortDeviceControl;
  DriverObject->MajorFunction[IRP_MJ_SCSI] =
    ScsiPortDispatchScsi;
  ...
}

ReactOS

V519 The 'pmsnName->SectionFileName.Length' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 294, 295. psapi psapi.c 295


static DWORD
InternalGetMappedFileName(....)
{
  ...
  pmsnName->SectionFileName.Length = 0;
  pmsnName->SectionFileName.Length = nBufSize;
   ...
}

ReactOS

V519 The 'ObjectName' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1129, 1130. ntoskrnl obname.c 1130


typedef WCHAR *PWCH;

NTSTATUS
ObQueryNameString(....)
{
  PWCH ObjectName;
  ...
  /* Directory without a name, we add "..." */
  ObjectName -= sizeof(L"...");
  ObjectName = L"...";
  break;
  ...
}

ReactOS

V519 The 'cols [1]' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 716, 719. cardlib cardwindow.cpp 719


NTSTATUS
CreateCdRomDeviceObject(....)
{
  ...
  cddata->XAFlags &= ~XA_USE_6_BYTE;
  cddata->XAFlags = XA_USE_READ_CD | XA_USE_10_BYTE;
  ...
}

ReactOS

V519 The 'cols [1]' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 716, 719. cardlib cardwindow.cpp 719


HPALETTE CardWindow::CreateCardPalette()
{
  ...
  //include button text colours
  cols[0] = RGB(0, 0, 0);
  cols[1] = RGB(255, 255, 255);

  //include the base background colour
  cols[1] = crBackgnd;

  //include the standard button colours...
  cols[3] = CardButton::GetHighlight(crBackgnd);
  cols[4] = CardButton::GetShadow(crBackgnd);
  cols[5] = CardButton::GetFace(crBackgnd);
  ...
}

Most likely this is what should be written here: cols[2] = crBackgnd;

IPP Samples

V519 The 'aInfo.allTime' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 336, 338. 3d-viewer 3dvmain.cpp 338


static DWORD WINAPI Decoder(void *p)
{
  ...
  aInfo.statIntercekt.statIntersectEye = 0.;
  aInfo.statIntercekt.statCPPPostPrc = 0.;
  aInfo.statIntercekt.statInner = 0.;
  aInfo.statIntercekt.statCPR = 0.;
  aInfo.allTime = 0.f;
  statCount = MAX_STAT_COUNT;
  aInfo.allTime = 0.;
  ...
}

Most likely this is what should be written here: aInfo.kdtTime = 0.;

IPP Samples

V519 The 'core_enc->m_pCurrentFrame->m_FrameNum' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1804, 1805. h264_enc umc_h264_video_encoder_tmpl.cpp.h 1805


Status H264ENC_MAKE_NAME(H264CoreEncoder_UpdateRefPicMarking)(
    void* state)
{
  ...
  // set frame_num to zero for this picture, for correct
  // FrameNumWrap
  core_enc->m_pCurrentFrame->m_FrameNum = 0;
  core_enc->m_pCurrentFrame->m_FrameNum = 0;
  ...
}

Chromium

V519 The 'load' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1346, 1350. yasm coff-objfmt.c 1350


static int
coff_helper_gasflags(....)
{
  ...
  case 'd':
      datasect = 1;
      load = 1;
      readonly = 0;
  case 'x':
      code = 1;
      load = 1;
      break;
  ...
}

The break operator is missing.

OGRE

V519 The 'tu' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 370, 372. OgreTerrain ogreterrainmaterialgeneratora.cpp 372


void TerrainMaterialGeneratorA::SM2Profile::addTechnique(....)
{
  ...
  TextureUnitState* tu;
  ...
  for (uint i = 0; i < numLayers; ++i)
  {
    // diffuse / specular
    tu = pass->createTextureUnitState(
                 terrain->getLayerTextureName(i, 0));
    // normal / height
    tu = pass->createTextureUnitState(
                 terrain->getLayerTextureName(i, 1));
  }
  ...
}

Doom 3

V519 The 'eof' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 216, 221. Game script_compiler.cpp 221


idCompiler::idCompiler() {
  ...
  eof = true;
  ...
  eof = false;
  ...
}

Mozilla Firefox

V519 The '* vp' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 711, 712. jsdbgapi.cpp 712


JS_PUBLIC_API(JSBool)
JS_GetValidFrameCalleeObject(JSContext *cx, JSStackFrame *fp,
                             jsval *vp)
{
    Value v;

    if (!Valueify(fp)->getValidCalleeObject(cx, &v))
        return false;
    *vp = v.isObject() ? v : JSVAL_VOID;
    *vp = v;
    return true;
}

Mozilla Firefox

V519 The 'event.refPoint.x' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 545, 546. puppetwidget.cpp 546


nsresult
PuppetWidget::DispatchPaintEvent()
{
  ...
  event.refPoint.x = dirtyRect.x;
  event.refPoint.x = dirtyRect.y;
  ...
}

Quake-III-Arena

V519 The 'numQuadCels' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1004, 1006. quake3 cl_cin.c 1006


static void setupQuad( long xOff, long yOff )
{
    ...
  numQuadCels  = (.....);
  numQuadCels += numQuadCels/4 + numQuadCels/16;
  numQuadCels += 64;
  numQuadCels  = (.....);
  numQuadCels += numQuadCels/4;
  numQuadCels += 64;
  ...
}

wxWidgets

V519 The 'm_isDirty' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 249, 250. core dragimgg.cpp 250


bool m_isDirty;
bool m_isShown;

bool wxGenericDragImage::BeginDrag(const wxPoint& hotspot,
                                   wxWindow* window,
                                   bool fullScreen,
                                   wxRect* rect)
{
  ...
  m_isDirty = false;
  m_isDirty = false;
  ...
}

Most likely this is what should be written here: m_isShown = false;

wxWidgets

V519 The 'rasDialParams.szCallbackNumber[0]' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 832, 833. core dialup.cpp 833


WCHAR szEntryName[ RAS_MaxEntryName + 1 ];
WCHAR szPhoneNumber[ RAS_MaxPhoneNumber + 1 ];
WCHAR szCallbackNumber[ RAS_MaxCallbackNumber + 1 ];

bool wxDialUpManagerMSW::Dial(const wxString& nameOfISP,
                              const wxString& username,
                              const wxString& password,
                              bool async)
{
  ...
  // default values for other fields
  rasDialParams.szPhoneNumber[0] = '\0';
  rasDialParams.szCallbackNumber[0] = '\0';
  rasDialParams.szCallbackNumber[0] = '\0';
  ...
}

Most likely this is what should be written here: rasDialParams.szEntryName[0] = '\0';

WinMerge

V519 The 'code' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 79, 80. Merge timesizecompare.cpp 80


int TimeSizeCompare::CompareFiles(int compMethod,
                                  const DIFFITEM &di)
{
  UINT code = DIFFCODE::SAME;
  ...
  if (di.left.size != di.right.size)
  {
    code &= ~DIFFCODE::SAME;
    code = DIFFCODE::DIFF;
  }
  ...
}

WinMerge

V519 The 'strParsed' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 339, 342. Merge filefilterhelper.cpp 342


String FileFilterHelper::ParseExtensions(
  const String &extensions) const
{
  ...
  strParsed = _T("^");
  strPattern = string_makelower(strPattern);
  strParsed = strPattern;
  ...
}

Trans-Proteomic Pipeline

V519 The 'pvalue' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 205, 214. tpplib asapcgiparser.cxx 214 (....)


void ASAPCGIParser::writeProteinRatio(ostream& os,
  proDataStrct* pro_ratio, const char* name)
{
  ...
  pvalue = (double)norm_->normalize(adj_inv_ratio);  <<<---

  double tmp[2];
  tmp[0] = adj_inv_ratio[0];
  tmp[1] = adj_inv_ratio[1];
  adj_inv_ratio[0] = 1/ tmp[0];
  adj_inv_ratio[1] = tmp[1]/(tmp[0]*tmp[0]);

  pvalue = (double)norm_->normalize(adjratio);       <<<---
  ...
}

D programming language

V519 The 'c2' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1860, 1861. cod2.c 1861


code *cdcond(elem *e,regm_t *pretregs)
{
  code *cc,*c,*c1,*cnop1,*c2,*cnop2;
  ....
  c2 = codelem(e22,&retregs,FALSE);
  c2 = cat(c1, fixresult(e22,retregs,pretregs));
  ....
}

Most likely this is what should be written here: c2 = cat(c2, ....);

OpenCV

V519 The 'ccp->sampgrdsubstepx' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 414, 415. jpc_enc.c 415


static jpc_enc_cp_t *cp_create(char *optstr, jas_image_t *image)
{
  ....
  ccp->sampgrdsubstepx = 0;
  ccp->sampgrdsubstepx = 0;
  ....
}

OpenCV

V519 The 'pass->lyrno' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 539, 540. jpc_t2enc.c 540


void jpc_init_t2state(jpc_enc_t *enc, int raflag)
{
  ....
  for (pass = cblk->passes; pass != endpasses; ++pass) {
    pass->lyrno = -1;
    pass->lyrno = 0;
  }
  ....
}

ReactOS

V519 The 'cddata->XAFlags' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 1290, 1291. cdrom.c 1291


NTSTATUS NTAPI
CreateCdRomDeviceObject(....)
{
  ....
  cddata->XAFlags &= ~XA_USE_6_BYTE;
  cddata->XAFlags = XA_USE_READ_CD | XA_USE_10_BYTE;
  ....
}

<< Return to list of all diagnostics

Static Code Analyzer for C/C++/C++11

Examples of errors detected by the V519 diagnostic

Crystal Space 3D SDK

Ultimate TCP/IP

XUIFramework

eMule

Lugaru

Wolfenstein 3D

IPP Samples

IPP Samples

IPP Samples

IPP Samples

IPP Samples

IT++

Newton Game Dynamics

QT

QT

QT

QT

QT

QT

QT

MySQL

CMake

Fireflies

Intel AMT SDK

Clang

Clang

Clang

ReactOS

ReactOS

ReactOS

ReactOS

ReactOS

ReactOS

ReactOS

ReactOS

IPP Samples

IPP Samples

Chromium

OGRE

Doom 3

Mozilla Firefox

Mozilla Firefox

Quake-III-Arena

wxWidgets

wxWidgets

WinMerge

WinMerge

Trans-Proteomic Pipeline

D programming language

OpenCV

OpenCV

ReactOS