Ами да няма защити Също така ако стринговете са празни, трябва да се вкара предварително 0х00 в string[0] на масива, щото няма да знае откъде да почне да цвъка. Ако е char * string = ""; то си е NULL терминирано.

Ами то не е до защити, просто начина по който е написано, с този while а и алгоритмично, преполага много строги данни и почти никакви отклонения, както към съдържанието, така и към дължината,не е много добра практика защото утре ще забравиш как си я написал тая функция, ще я бутнеш в някой друг проект и ще се чудиш що се прецаква нещото. Отделно че изискването за нула е малко ... трябва да пълниш масива с нули, или трябва винаги да е строго дефинирана дължина и да си сигурен че поне последния елемент е 0.