hdf5 icon indicating copy to clipboard operation
hdf5 copied to clipboard

Replace all sprintf with safer snprintf

Open seanm opened this issue 2 years ago • 1 comments

Even after https://github.com/HDFGroup/hdf5/pull/1815 there are 16 remaining uses of HDsprintf() left. They were a bit too tricky for me to confidently change as I don't know the codebase well.

seanm avatar Jun 23 '22 17:06 seanm

@jhendersonHDF

seanm avatar Jun 23 '22 17:06 seanm

I took another stab at this, and was disappointed to see there are more sprintf than there used to be.

Anyway, I fixed more of them. After this there is only 1 left!

seanm avatar Feb 11 '24 23:02 seanm

Thanks @seanm, I'd forgotten about this; will review.

jhendersonHDF avatar Feb 12 '24 17:02 jhendersonHDF

All done. Yay!

derobins avatar Mar 07 '24 11:03 derobins