Page MenuHomeSoftware Heritage

Fix connection leaks in revision_add and directory_add.
ClosedPublic

Authored by vlorentz on Apr 15 2019, 12:43 PM.

Diff Detail

Repository
rDSTO Storage manager
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vlorentz created this revision.Apr 15 2019, 12:43 PM

could you explain in the commit message why this solves a cnx leak?

douardda requested changes to this revision.Apr 15 2019, 1:27 PM
This revision now requires changes to proceed.Apr 15 2019, 1:27 PM
vlorentz updated this revision to Diff 4576.Apr 15 2019, 1:58 PM

Better commit message.

vlorentz updated this revision to Diff 4578.Apr 15 2019, 2:22 PM

Fix commit message to match the actual source of the error.

ardumont accepted this revision.Apr 15 2019, 2:30 PM
douardda accepted this revision.Apr 15 2019, 4:39 PM
This revision is now accepted and ready to land.Apr 15 2019, 4:39 PM
This revision was landed with ongoing or failed builds.Apr 16 2019, 10:27 AM
This revision was automatically updated to reflect the committed changes.