EPICS Base

Code review comment for ~anj/epics-base/+git/base-7.0:fix-1824277

  1. Git
  2. lp:~anj/epics-base/+git/base-7.0
  3. fix-1824277
  4. Merge into 7.0
Revision history for this message
mdavidsaver (mdavidsaver) wrote :

Looking through the new code for dbPutFieldLink(), I'm suspicious that the allocated 'pdbaddr' would be leaked in some (probably error) cases. This may already be possible. Good practice might be to set 'pdbaddr=NULL' when it is consumed by dbAddLink(), then add a 'free(pdbaddr)' to the cleanup step. Also need to make dbAddLink() free it on error.

review: Needs Fixing
Reply

« Back to merge proposal