You are viewing a plain text version of this content. The canonical link for it is here.
Posted to c-dev@xerces.apache.org by GitBox <gi...@apache.org> on 2021/12/20 19:15:55 UTC

[GitHub] [xerces-c] rouault opened a new pull request #45: DFAContentModel::buildDFA(): correctly zero-initialize fFollowList

rouault opened a new pull request #45:
URL: https://github.com/apache/xerces-c/pull/45


   Due to a copy&paste issue, the intended zero-initialization of
   fFollowList wasn't done (copy&paste issue), and thus in case of
   OutOfMemory exception when initializing the array, the memory freeing in
   cleanup() could access uninitialized elements.
   
   Follow-up of https://github.com/apache/xerces-c/pull/40 / a65990d79d3fc333d7481f010da4e165a88b6cb3
   
   Fixes GDAL's https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=42636


-- 
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


[GitHub] [xerces-c] rleigh-codelibre commented on pull request #45: [XERCESC-2235] DFAContentModel::buildDFA(): correctly zero-initialize fFollowList

Posted by GitBox <gi...@apache.org>.
rleigh-codelibre commented on pull request #45:
URL: https://github.com/apache/xerces-c/pull/45#issuecomment-1065923208


   Yes, this looks fine to me.  Thanks.


-- 
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


[GitHub] [xerces-c] rouault commented on pull request #45: [XERCESC-2235] DFAContentModel::buildDFA(): correctly zero-initialize fFollowList

Posted by GitBox <gi...@apache.org>.
rouault commented on pull request #45:
URL: https://github.com/apache/xerces-c/pull/45#issuecomment-1063792952


   @rleigh-codelibre would you mind merging ? This is a trivial bug fix.


-- 
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


[GitHub] [xerces-c] rleigh-codelibre merged pull request #45: [XERCESC-2235] DFAContentModel::buildDFA(): correctly zero-initialize fFollowList

Posted by GitBox <gi...@apache.org>.
rleigh-codelibre merged pull request #45:
URL: https://github.com/apache/xerces-c/pull/45


   


-- 
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