Seems fine for automatic processing, but perhaps it will create each disulfide twice? This may not be a problem though, but you should check with a system where you also define the disulfides yourself.
A semi-automatic way of doing this for systems that you are not familiar with is to first get all the CYS SG-SG distances shorter than 2.5Å:
coor dist sele atom * cys sg end sele atom * cys sg end cut 2.5
and then use the resulting output to setup patches for the handful of real disulfides.
Edited by lennart (11/21/18 04:05 PM)