Си — Говнокод.ру

Си / Говнокод #2848

+96

1
2
3
4
5
6
7
8

char str[STR_LEN + 1]; 

strncpy(str, inpust_str, STR_LEN);
xxx[STR_LEN] = '\0';

str[ strlen(str) ] = ':';
str[ strlen(str) + 1 ] = '1';
str[ strlen(str) + 2 ] = '\0';

Попытка добавить ":1" к строке.

Первым же "str[ strlen(str) ] = ':';" гробим нулевой символ в конце строки. Дальнейшие вызовы "strlen" - просто фигня какая-то. Ну и портим память.

benderlog, 23 Марта 2010

Комментарии (7)

Си / Говнокод #2829

+144.8

1
2
3
4
5
6
7
8

1. #include <stdio.h>
   2.  
   3. void main()
   4. {
   5.  int i = 5;
   6.  i = ++i + ++i;
   7.  printf("%d\n",i);
   8. }

http://habrahabr.ru/blogs/crazydev/88185/

stepushyn, 19 Марта 2010

Комментарии (1)

Си / Говнокод #2823

+144.8

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26

// C:\Program Files\Microsoft Visual Studio 9.0\VC\crt\src\dbghook.c

int __cdecl _CrtDefaultAllocHook(
        int nAllocType,
        void * pvData,
        size_t nSize,
        int nBlockUse,
        long lRequest,
        const unsigned char * szFileName,
        int nLine
        )
{
        return 1; /* allow all allocs/reallocs/frees */
}


// C:\Program Files\Microsoft Visual Studio 9.0\VC\crt\src\dbgheap.c

extern "C" _CRTIMP int __cdecl _CrtIsValidPointer(
        const void * pv,
        unsigned int nBytes,
        int bReadWrite
        )
{
        return (pv != NULL);
}

вот, в Microsoft Visual Studio 2008 нашёл такие вот перлы...

glook, 19 Марта 2010

Комментарии (11)

Си / Говнокод #2816

+137.6

1
2
3
4
5
6
7
8

for( unsigned int i = 0; i < strlen(str); i++)
{
    if ( str[i] == '-')
    {
        str[i] = '\0';
        break;
    }
}

'strlen' в условии цикла - это вычисление инварианта в теле цикла.
Вообще этот цикл реально можно заменить одним вызовом 'strchr'

benderlog, 18 Марта 2010

Комментарии (9)

Си / Говнокод #2770

+138.9

1
2
3
4

res =  Wlxtbl.WlxLoggedOutSAS(pWlxContext,dwSasType,pAuthenticationId,pLogonSid,pdwOptions,phToken,pNprNotifyInfo,pProfile);
switch(res) {
	case WLX_SAS_ACTION_LOGON:
		if(res == WLX_SAS_ACTION_LOGON) {

Пишу свою gina.dll, только что заметил вот это...

zitzy, 12 Марта 2010

Комментарии (17)

Си / Говнокод #2751

+106.7

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72

//Разбор принятых данных

  //===========================================================================
  //IP-адрес сервера
  //===========================================================================
   for(i=0;i<16;i++)
	 {
	  if(CtrlData[i] == ',')
		break;
	  else
		buf[i] = CtrlData[i];
	 }

   //разбираем первый байт адреса
   if(buf[1]=='.')      //вариант 1.
	 {
	  ip_buf[0] = ' ';
	  ip_buf[1] = ' ';
	  ip_buf[2] = buf[0];
	  ip_buf[3] = buf[1];
	  ip_ptr = 2;
	 }
   else if(buf[2]=='.') //вариант 11.
	 {
	  ip_buf[0] = ' ';
	  ip_buf[1] = buf[0];
	  ip_buf[2] = buf[1];
	  ip_buf[3] = buf[2];
	  ip_ptr = 3;
	 }
   else if(buf[3]=='.') //вариант 111.
	 {
	  for(i=0;i<4;i++)
		ip_buf[i] = buf[i];
	  ip_ptr = 4;
	 }
   else
	 {
	  MessageBox(0, "Память устройства или файл содержит неверные данные","Ошибка", MB_ICONERROR|MB_TOPMOST|MB_OK|MB_TASKMODAL);
	  return 1;
	 }

   //разбираем второй байт адреса
   if(buf[ip_ptr+1]=='.')      //вариант 1.
	 {
	  ip_buf[4] = ' ';
	  ip_buf[5] = ' ';
	  ip_buf[6] = buf[ip_ptr];
	  ip_buf[7] = buf[ip_ptr+1];
	  ip_ptr += 2;
	 }
   else if(buf[ip_ptr+2]=='.') //вариант 11.
	 {
	  ip_buf[4] = ' ';
	  ip_buf[5] = buf[ip_ptr];
	  ip_buf[6] = buf[ip_ptr+1];
	  ip_buf[7] = buf[ip_ptr+2];
	  ip_ptr += 3;
	 }
   else if(buf[ip_ptr+3]=='.') //вариант 111.
	 {
	  ip_buf[4] = buf[ip_ptr];
	  ip_buf[5] = buf[ip_ptr+1];
	  ip_buf[6] = buf[ip_ptr+2];
	  ip_buf[7] = buf[ip_ptr+3];
	  ip_ptr += 4;
	 }
   else
	 {
	  MessageBox(0, "Память устройства или файл содержит неверные данные","Ошибка", MB_ICONERROR|MB_TOPMOST|MB_OK|MB_TASKMODAL);
	  return 1;
	 }

парсинг второй половины адреса сюда полностью не влез, но там ничего нового, я вас уверяю =)

angry C nerd, 11 Марта 2010

Комментарии (17)

Си / Говнокод #2746

+104.7

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58

/*==================================================================================*/
/* Преобразование целого числа в ASCII												*/
/*==================================================================================*/
unsigned char IntToStr (unsigned long Value,unsigned char *pBuf,unsigned char n,unsigned char Zeros)
{
// Локальные переменные
	unsigned char 	j = 0;
	unsigned long 	dec;
	uldiv_t			Tmp;
// Программа
// Перевод ASCII в значение
	while (n--)
	{
		switch(n)
		{
			case 0:
				dec = 1;
			break;
			case 1:
				dec = 10;
			break;
			case 2:
				dec = 100;
			break;
			case 3:
				dec = 1000;
			break;
			case 4:
				dec = 10000;
			break;
			case 5:
				dec = 100000;
			break;
			case 6:
				dec = 1000000;
			break;
			case 7:
				dec = 10000000;
			break;
			case 8:
				dec = 100000000;
			break;
			case 9:
				dec = 1000000000;
		}
		Tmp = uldiv(Value,dec);
	// Остаток
		Value = Tmp.rem;
	// Ограничение
		if (Tmp.quot > 9)
			Tmp.quot = 9;
	// Проверка целой части
		if (Zeros || j || Tmp.quot || (dec == 1))
			*(pBuf + j++) = Hex_Table[Tmp.quot];
	}
	*(pBuf + j) = '\0';
	return j;
}

без комментариев.

angry C nerd, 10 Марта 2010

Комментарии (13)

Си / Говнокод #2744

+100.3

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27

int main()
{
	puts("<?php\n");
	int ch, xit, i, j;
	for(j=0;j<4;j++) {
	printf("$weather_date[%d]='", j);
	xit=1;
	while(xit) {
	do { ch = getc(stdin); } while((ch != EOF)&&(ch != '<'));
	if(ch == EOF) return 1;
	h = getc(stdin);
	if(ch == EOF) return 1;
	if(ch == 't') {
	ch = getc(stdin);
	if(ch == EOF) return 1;
	if(ch == 'd') {
	do { ch = getc(stdin); } while((ch != EOF)&&(ch!='>'));
	if(ch == EOF) return 1;
	do { ch = getc(stdin); if((ch != EOF)&&(ch != '<')) printf("%c", ch); } while((ch != EOF)&&(ch != '<'));
	if(ch == EOF) return 1;
	printf("';\n$weather_time[%d]='", j);
		for(i=0;((ch = getc(stdin)) != EOF)&&(i<2); i++);
	if(ch == EOF) return 1;
	do { ch = getc(stdin); if((ch != EOF)&&(ch != '<')) printf("%c", ch); } while((ch != EOF)&&(ch != '<'));
	if(ch == EOF) return 1;
	puts("';\n");
	xit = 0;

по крону генерится php-скрипт, прогой написанной на сях)

xXx_totalwar, 10 Марта 2010

Комментарии (14)

Си / Говнокод #2738

+138.1

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21

unsigned char SIM300_Reboot(void)
{
    // проверим статус модема
    if (!pinSTATUS)             // если STATUS = 0 (модем выключен)
    {
        while (1)               //если не включается-включаем до посинения
        {
            if (!(Modem_ON()))
                return 0;
        }

    } else                      // если STATUS = 1 (модем включен)
    {
        while (1)               //если не выключается - выключаем до посинения
        {
            if ((!Modem_OFF()) && (!Modem_ON()))
                return 0;
        }

    }
}

тяжёлые будни программиста embedded-систем.
дополнительный аромат: упомянутые функции возвращают 0 в случае успешного выполнения, а не наоборот, как можно было бы подумать при виде if (!some_func(...))

angry C nerd, 09 Марта 2010

Комментарии (10)

Говнокод: по колено в коде.