Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Revert "update dmlc-core for security reason"#12172

Closed
hcho3 wants to merge 15 commits intoapache:masterfrom
hcho3:revert-12129-update_dmlc_core
Closed

Revert "update dmlc-core for security reason"#12172
hcho3 wants to merge 15 commits intoapache:masterfrom
hcho3:revert-12129-update_dmlc_core

Conversation

@hcho3
Copy link
Copy Markdown
Contributor

@hcho3 hcho3 commented Aug 15, 2018

Reverts #12129

This is to fix double free memory error in #12139. The memory error has been inside CSVIter but remained undetected until a recent modification of CSVParser.

hcho3 added 15 commits August 14, 2018 17:04
@hcho3 hcho3 closed this Aug 15, 2018
@hcho3 hcho3 deleted the revert-12129-update_dmlc_core branch August 15, 2018 00:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant