[prev in list] [next in list] [prev in thread] [next in thread] 

List:       xerces-c-dev
Subject:    =?utf-8?q?=5BGitHub=5D_=5Bxerces-c=5D_johnjamesmccann_commented_on_pull_requ?= =?utf-8?q?est_=2347=3
From:       GitBox <git () apache ! org>
Date:       2022-01-25 17:15:00
Message-ID: 164313090018.29368.7867331695648160219.asfpy () gitbox ! apache ! org
[Download RAW message or body]


johnjamesmccann commented on pull request #47:
URL: https://github.com/apache/xerces-c/pull/47#issuecomment-1021204613


   Hi Rouault,
   
   I cant see how my changes could cause a test regression, all my change does is \
delete 4 smart pointers that are not used in the code base.  
   Kind regards
   
   John
   
   From: Even Rouault ***@***.***>
   Sent: 23 January 2022 15:34
   To: apache/xerces-c ***@***.***>
   Cc: McCann, John (DI SW PE OT IO PP) ***@***.***>; Mention ***@***.***>
   Subject: Re: [apache/xerces-c] [XERCESC-2188] Fix potential double-free in usage \
of ReaderMgr::pushReader() (PR #47)  
   
   Seeing now https://issues.apache.org/jira/projects/XERCESC/issues/XERCESC-2188 , I \
see my approach is close to a suggestion of \
https://issues.apache.org/jira/browse/XERCESC-2188?focusedCommentId=17055399&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17055399
  
   —
   Reply to this email directly, view it on \
GitHub<https://github.com/apache/xerces-c/pull/47#issuecomment-1019508927>, or \
unsubscribe<https://github.com/notifications/unsubscribe-auth/AXMN5WD5SDILV22VLP5YNFLUXQNXNANCNFSM5MTPEBCQ>.
  Triage notifications on the go with GitHub Mobile for \
iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> \
or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
  You are receiving this because you were mentioned.Message ID: \
***@***.******@***.***>>  
   -----------------
   Siemens Industry Software Limited is a limited company registered in England and \
Wales.  Registered number: 3476850.
   Registered office: Pinehurst 2, Pinehurst Road, Farnborough, Hampshire, GU14 7BF.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: c-dev-unsubscribe@xerces.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscribe@xerces.apache.org
For additional commands, e-mail: c-dev-help@xerces.apache.org


[prev in list] [next in list] [prev in thread] [next in thread]