Копирование строк. Правильный ли код?

[LinuxUser]

Я использую следующий код для копирования строк:

#include <string.h>
#include <stdlib.h>
#include <stdio.h>

int main()
	{char len;
	char s1[1024]="t1 t2";
	unsigned short length=strlen(s1)+1;
	char *s2=(char *)malloc(len);
	strcpy(s2,s1);
	printf("length=%i\ns1=%s\ns2=%s\n",len,s1,s2);
	free((void *)s2);
	s2=0;
	return 0;
	}

Есть ли тут ошибки, недочёты?

[Alek86]

вот тут
unsigned short length=strlen(s1)+1;
char *s2=(char *)malloc(len);

[Carbon]

LinuxUser, а почему ты решил, что в твоей программе есть косяки? Не работает разве?

А в строке:
unsigned short length=strlen(s1)+1;

нет ничего страшного. Если, конечно, у тебя строка будет меньше 65535.

[merax]

А как же насчет защищенного кода ???

Ведь в таком коде потенциально много ошибок которые делают этот код не безопасным!!! Переполнение буфера... грубая очистка памяти.... и т.д.

[Alek86]

Carbon, merax а присмотреться?

[Alek86]

вообще ошибка или спецальная или тупая....

[Carbon]

А я сразу не заметил.

char len; убери, чтобы не смущало.

[merax]

Сообщение от Carbon

А я сразу не заметил.

char len; убери, чтобы не смущало.

Аналогично блин...

[LinuxUser]

Извеняюсь, чуток не тот код скопировал.
Изначално было length, а потом я его заменил на len.

Вот правильный код:

#include <string.h>
#include <stdlib.h>
#include <stdio.h>

int main()
	{char s1[1024]="t1 t2";
	unsigned short len=strlen(s1)+1;
	char *s2=(char *)malloc(len);
	strcpy(s2,s1);

	printf("len=%i\ns1=%s\ns2=%s\n",len,s1,s2);

	free((void *)s2);
	s2=0;
	return 0;
	}

[Alek86]

если ты есть студент, то с пивом покатит

на работу С++ программистом с таким кодом я б тя не взял

учи операторы operator new, operator delete, а также stl (можно еще обработку исключений и потоковую безопасность, но это уж для уровня "профи")