)]}'
{"id":"openstack%2Fliberasurecode~385240","triplet_id":"openstack%2Fliberasurecode~master~I7bfc481c7a9bbc64288760df2dc6053c57657b41","project":"openstack/liberasurecode","branch":"master","topic":"reconstruct","hashtags":[],"change_id":"I7bfc481c7a9bbc64288760df2dc6053c57657b41","subject":"Fix reconstruct to return an error when memory allocation failed","status":"MERGED","created":"2016-10-12 02:39:09.000000000","updated":"2016-11-24 00:18:09.000000000","submitted":"2016-11-24 00:18:08.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2ddc6e4bf5a168ceb01c93444ddf7a7c8fb81c54","_number":385240,"virtual_id_number":385240,"owner":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2016-11-24 00:18:08.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},{"value":0,"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-11-24 00:14:37.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},{"value":2,"date":"2016-11-10 19:03:48.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-11-24 00:14:37.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},{"value":0,"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-11-10 19:03:48.000000000","updated_by":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"reviewer":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"state":"REVIEWER"},{"updated":"2016-11-24 00:14:37.000000000","updated_by":{"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},"reviewer":{"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},"state":"REVIEWER"},{"updated":"2016-11-24 00:18:08.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"6c7bf6130ed766ff92c1bf59e696324bf857c81a","author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"date":"2016-10-12 02:39:09.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"adda0e32db35336cd61a424c3722b79ecaacc18c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-10-12 02:44:06.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-liberasurecode-unit http://logs.openstack.org/40/385240/1/check/gate-liberasurecode-unit/1260469/ : SUCCESS in 4m 43s","accounts_in_message":[],"_revision_number":1},{"id":"dcb39c213acb997231ac49fffc658549e9dcce08","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2016-10-31 21:07:04.000000000","message":"Patch Set 1: Code-Review+1\n\nSeems to make sense. Is there any good way for us to test memory allocation errors like this?","accounts_in_message":[],"_revision_number":1},{"id":"323154363a7e98696bdd48cbb48c82417006f75f","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2016-11-10 19:03:48.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"a21f16fd3c5fe9743a2b45a4aa9fcfdc883018ed","author":{"_account_id":7485,"name":"Tushar Gohad","email":"tushar.gohad@intel.com","username":"tsg"},"date":"2016-11-24 00:14:37.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"b3b165d8013f6e888b387314aaef32f83025633d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-24 00:14:47.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"3d08e1b906bf38bea7e1c285d99f10181c509f68","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-24 00:18:08.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-liberasurecode-unit http://logs.openstack.org/40/385240/1/gate/gate-liberasurecode-unit/fcc0a7c/ : SUCCESS in 3m 19s","accounts_in_message":[],"_revision_number":1},{"id":"d7b0e9667160d8f5c3e656763e0d2546390742a4","date":"2016-11-24 00:18:09.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"10d6fa5f02dd9a3a7bd6cd00ce79b78a1446d7d0","revisions":{"10d6fa5f02dd9a3a7bd6cd00ce79b78a1446d7d0":{"kind":"REWORK","_number":1,"created":"2016-10-12 02:39:09.000000000","uploader":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"ref":"refs/changes/40/385240/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/liberasurecode","ref":"refs/changes/40/385240/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/liberasurecode refs/changes/40/385240/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/liberasurecode refs/changes/40/385240/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/liberasurecode refs/changes/40/385240/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/liberasurecode refs/changes/40/385240/1"}}},"commit":{"parents":[{"commit":"cb0daba975dfb3b7bffe402d7aa73fac19180f73","subject":"Add get_by_desc ret val handling to get_fragment_size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/liberasurecode/commit/cb0daba975dfb3b7bffe402d7aa73fac19180f73"}]}],"author":{"name":"Kota Tsuyuzaki","email":"tsuyuzaki.kota@lab.ntt.co.jp","date":"2016-10-12 02:35:04.000000000","tz":-420},"committer":{"name":"Kota Tsuyuzaki","email":"tsuyuzaki.kota@lab.ntt.co.jp","date":"2016-10-12 02:35:04.000000000","tz":-420},"subject":"Fix reconstruct to return an error when memory allocation failed","message":"Fix reconstruct to return an error when memory allocation failed\n\nAs like current code, if we do go jump to just \"out\" section when\nmemory allocation failed, liberaurecode/PyECLib will report it as\nsuccess. However, in fact, the returned binary is not the one\nreconstructed. This patch fixes liberasurecode_reconstruct to return\nerror code (ENOMEM) if either malloc or memset for working space is\nfailed.\n\nChange-Id: I7bfc481c7a9bbc64288760df2dc6053c57657b41\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/liberasurecode/commit/10d6fa5f02dd9a3a7bd6cd00ce79b78a1446d7d0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/liberasurecode/commit/10d6fa5f02dd9a3a7bd6cd00ce79b78a1446d7d0"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}