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

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

+132

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

bool bitmap_to_24bit_string(bitmap *bmp, char **str, uint32_t *len)
{
    if (!bmp || !bmp->pixels)
    return false;
    int I, J;
    uint32_t size = ((bmp->width * 24 + 31) / 32) * 4 * bmp->height;
    rgb24 *pixels = malloc(size);
    if (pixels)
    {
        for (I = 0; I < bmp->height; ++I)
        {
            for (J = 0; J < bmp->width; ++J)
            {
                pixels[I * bmp->width + J].b = bmp->pixels[I * bmp->width + J].r;
                pixels[I * bmp->width + J].g = bmp->pixels[I * bmp->width + J].g;
                pixels[I * bmp->width + J].r = bmp->pixels[I * bmp->width + J].b;
            }
        }
        uint32_t destlen = compressBound(size);
        *str = malloc(destlen);
        if (*str)
        {
            if (compress((Bytef *)*str, (uLongf *)&destlen, (Bytef *)pixels, size) == Z_OK)
            {
                free(pixels);
                pixels = NULL;
                char *b64str;
                uint32_t b64_len;
                if (base64encode((const uint8_t *)*str, destlen, &b64str, &b64_len))
                {
                    free(*str);
                    *str = b64str;
                    *len = b64_len + 2;
                    b64str = malloc(*len);
                    if (b64str)
                    {
                        b64str[0] = 'm';
                        strncpy(&b64str[1], *str, b64_len);
                        free(*str);
                        *str = b64str;
                        (*str)[b64_len + 1] = '';
                        return true;
                    }
                }
            }
            free(*str);
            *len = 0;
            *str = NULL;
        }
        free(pixels);
    }
    return false;
}

Ещё подкину в общую копилку

Cynicrus, 01 Декабря 2014

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

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

+136

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

char *stringFromDTM(MDTM *dtm)
{
    if (dtm->count < 1)
    return "";
    uint32_t size = (sizeof(MDTMPoint) + sizeof(uint32_t)) * dtm->count;
    void *data = calloc(1, size);
    uint32_t *ptr = data;
    *(ptr++) = size;
    uint32_t index;
    for (index = 0; index < dtm->count; index++)
    *(ptr++) = dtm->points[index].x;
    for (index = 0; index < dtm->count; index++)
    *(ptr++) = dtm->points[index].y;
    for (index = 0; index < dtm->count; index++)
    *(ptr++) = dtm->points[index].color;
    for (index = 0; index < dtm->count; index++)
    *(ptr++) = dtm->points[index].tol;
    for (index = 0; index < dtm->count; index++)
    *(ptr++) = dtm->points[index].size;
    for (index = 0; index < dtm->count; index++)
    *(ptr++) = dtm->points[index].x;
    bool *bptr = (bool *)ptr;
    for (index = 0; index < dtm->count; index++)
    *(bptr++) = dtm->points[index].bad;
    uint32_t len = compressBound(size);
    char *buffer = malloc(len);
    if (compress((Bytef *)buffer, (uLongf *)&len, data, size) == Z_OK)
    {
        free(data);
        char *compressed = malloc(len + sizeof(uint32_t));
        *((uint32_t *)(compressed)) = len;
        strcpy(compressed + sizeof(uint32_t), buffer);
        free(buffer);
    }
    free(buffer);
    free(data);
    return "";
}

Я так и не раскурил, почему так, а не иначе.

Cynicrus, 01 Декабря 2014

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

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

+133

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

if (*wpPat == L'\\')
    {
      wpCharStart=wpPat;
      if (++wpPat >= wpMaxPat) goto Error;

      if (*wpPat == L'x')
      {
        if (++wpPat >= wpMaxPat) goto Error;

        if (*wpPat == L'{')
        {
          wpStrTmp=++wpPat;
          for (;;)
          {
            if (wpPat >= wpMaxPat) goto Error;
            if (*wpPat++ == L'}') break;
          }
          if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
          {
            nPatChar=(int)hex2decW(wpStrTmp, (wpPat - 1) - wpStrTmp);
            if (nPatChar == -1)
            {
              wpPat=wpStrTmp;
              goto Error;
            }
            if (nPatChar <= MAXWORD)
              lpREGroupItem->nGroupLen+=1;
            else
              lpREGroupItem->nGroupLen+=2;
          }
        }
        else
        {
          if (wpPat + 2 > wpMaxPat)
            goto Error;
          wpPat+=2;
          if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
            ++lpREGroupItem->nGroupLen;
        }
      }
      else if (*wpPat == L'u')
      {
        if (wpPat + 5 > wpMaxPat)
          goto Error;
        wpPat+=5;
        if (lpREGroupItem->nGroupLen != -1 && !bClassOpen)
          ++lpREGroupItem->nGroupLen;
      }

Регулярные велосипеды.
Akelpad.

gost, 25 Ноября 2014

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

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

+133

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

if (wParam == MI_CMDLINEBEGIN)
          return xstrcpyW((void *)lParam, wpCmdLineBegin);
        if (wParam == MI_CMDLINEEND)
          return xstrcpyW((void *)lParam, wpCmdLineEnd);
        if (wParam == MI_SHOWMODIFY)
          return moCur.dwShowModify;
        if (wParam == MI_STATUSPOSTYPE)
          return moCur.dwStatusPosType;
        if (wParam == MI_STATUSUSERFORMAT)
          return xstrcpynW((void *)lParam, moCur.wszStatusUserFormat, MAX_PATH);
        if (wParam == MI_WORDBREAKCUSTOM)
          return moCur.dwWordBreakCustom;
        if (wParam == MI_PAINTOPTIONS)
          return moCur.dwPaintOptions;
        if (wParam == MI_EDITSTYLE)
          return moCur.dwEditStyle;
        if (wParam == MI_RICHEDITCLASS)
          return moCur.bRichEditClass;
        if (wParam == MI_AKELADMINRESIDENT)
          return moCur.bAkelAdminResident;
        if (wParam == MI_DATELOGFORMAT)
          return xstrcpynW((void *)lParam, moCur.wszDateLogFormat, 128);
        if (wParam == MI_DATEINSERTFORMAT)
          return xstrcpynW((void *)lParam, moCur.wszDateInsertFormat, 128);
        if (wParam == MI_AKELUPDATEROPTIONS)
          return xstrcpynW((void *)lParam, moCur.wszAkelUpdaterOptions, MAX_PATH);
        if (wParam == MI_URLCOMMAND)
          return xstrcpynW((void *)lParam, moCur.wszUrlCommand, MAX_PATH);
        if (wParam == MI_TABNAMEFIND)
          return xstrcpynW((void *)lParam, moCur.wszTabNameFind, MAX_PATH);
        if (wParam == MI_TABNAMEREP)
          return xstrcpynW((void *)lParam, moCur.wszTabNameRep, MAX_PATH);
        if (wParam == MI_ONTOP)
          return moCur.bOnTop;
        if (wParam == MI_STATUSBAR)
          return moCur.bStatusBar;
        if (wParam == MI_KEEPSPACE)
          return moCur.bKeepSpace;
        if (wParam == MI_WATCHFILE)
          return moCur.bWatchFile;
        if (wParam == MI_SAVETIME)
          return moCur.bSaveTime;
        if (wParam == MI_SINGLEOPENFILE)
          return moCur.bSingleOpenFile;
        if (wParam == MI_SINGLEOPENPROGRAM)
          return moCur.dwSingleOpenProgram;
        if (wParam == MI_TABOPTIONSMDI)
          return moCur.dwTabOptionsMDI;
        if (wParam == MI_EXECUTECOMMAND)

И такого больше сотни строк.
Akelpad.

gost, 25 Ноября 2014

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

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

+132

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

LRESULT CALLBACK MainProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
  if (uMsg == WM_CREATE)
  {
    int *lpSBParts;
    int iBorders[3];
    CLIENTCREATESTRUCT ccs;
    DWORD dwClassStyle;
    RECT rcRect;
    HMENU hMenu;

    hMainWnd=hWnd;
    hPopupEdit=GetSubMenu(hPopupMenu, MENU_POPUP_EDIT);
    hPopupView=GetSubMenu(hPopupMenu, MENU_POPUP_VIEW);
    hPopupCodepage=GetSubMenu(hPopupMenu, MENU_POPUP_CODEPAGE);
    hPopupOpenCodepage=GetSubMenu(hPopupCodepage, MENU_POPUP_CODEPAGE_OPEN);
    hPopupSaveCodepage=GetSubMenu(hPopupCodepage, MENU_POPUP_CODEPAGE_SAVE);
    hPopupHeadline=GetSubMenu(hPopupMenu, MENU_POPUP_HEADLINE);

    hMenu=GetSubMenu(hMainMenu, MENU_FILE_POSITION);
    hMenuRecentFiles=GetSubMenu(hMenu, MENU_FILE_RECENTFILES_4X);
    hMenu=GetSubMenu(hMainMenu, MENU_VIEW_POSITION);
    hMenuLanguage=GetSubMenu(hMenu, MENU_VIEW_LANGUAGE_4X);

Плотная простынка, спать можно.
Akelpad.

gost, 25 Ноября 2014

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

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

+132

01
02
03
04
05
06
07
08
09
10
11

static inline uint64_t eight_bytes_to_uint64(uint8_t *buf) {
    return (uint64_t) four_bytes_to_uint32(buf) << 32 | four_bytes_to_uint32(buf + sizeof(uint32_t));
}

static inline uint32_t four_bytes_to_uint32(uint8_t *buf) {
    return (uint32_t) two_bytes_to_uint16(buf) << 16 | two_bytes_to_uint16(buf + sizeof(uint16_t));
}

static inline uint16_t two_bytes_to_uint16(uint8_t *buf) {
    return buf[0] << 8 | buf[1];
}

Чё-то вспомнились Schemeовые времена и ФП.

Buffer overflow исключён на 146%.

codemonkey, 25 Ноября 2014

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

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

+134

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

//// AEM_* AkelEdit control messages

  if (uMsg >= WM_USER)
  {
    if (uMsg >= AEM_CANUNDO)
      goto CanUndo;

    //Text retrieval and modification
    if (uMsg == AEM_EXSETTEXTA)
    {
      AESETTEXTA *at=(AESETTEXTA *)lParam;

      return AE_SetTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
    }
    if (uMsg == AEM_EXSETTEXTW)
    {
      AESETTEXTW *at=(AESETTEXTW *)lParam;

      return AE_SetText(ae, at->pText, at->dwTextLen, at->nNewLine, FALSE);
    }
    if (uMsg == AEM_SETTEXTA)
    {
      return AE_SetTextAnsi(ae, CP_ACP, (char *)lParam, wParam, AELB_ASINPUT);
    }
    if (uMsg == AEM_SETTEXTW)
    {
      return AE_SetText(ae, (wchar_t *)lParam, wParam, AELB_ASINPUT, FALSE);
    }
    if (uMsg == AEM_APPENDTEXTA)
    {
      AEAPPENDTEXTA *at=(AEAPPENDTEXTA *)lParam;

      AE_AppendTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
      return 0;
    }
    if (uMsg == AEM_APPENDTEXTW)
    {
      AEAPPENDTEXTW *at=(AEAPPENDTEXTW *)lParam;

      AE_AppendText(ae, at->pText, at->dwTextLen, at->nNewLine);
      return 0;
    }
    if (uMsg == AEM_REPLACESELA)
    {
      AEREPLACESELA *rs=(AEREPLACESELA *)lParam;

      AE_ReplaceSelAnsi(ae, rs->nCodePage, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
      return 0;
    }
    if (uMsg == AEM_REPLACESELW)
    {
      AEREPLACESELW *rs=(AEREPLACESELW *)lParam;

      AE_ReplaceSel(ae, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
      return 0;
    }
    if (uMsg == AEM_GETTEXTRANGEA)
    {
      AETEXTRANGEA *tr=(AETEXTRANGEA *)lParam;

      return AE_GetTextRangeAnsi(ae, tr->nCodePage, tr->lpDefaultChar, tr->lpUsedDefChar, &tr->cr.ciMin, &tr->cr.ciMax, tr->pBuffer, tr->dwBufferMax, tr->nNewLine, tr->bColumnSel, tr->bFillSpaces);
    }

И далее около сотни if'ов в таком духе.
AkelPad

gost, 25 Ноября 2014

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

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

+133

1
2
3
4
5
6
7
8
9

int size;

size = EXPR;

if (size > INT_MAX || size <= 0) {
    return NULL;
}

// ...

Ндя. Семь лет уже. Теперь всё понятно...

bot, 23 Ноября 2014

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

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

+139

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

SGELISTENTRY *sgeListAdd(SGELIST *l, const char *id, void *data) {
   SGELISTENTRY *ret;

   sgeNew(ret, SGELISTENTRY);
   l->numberOfEntries++;
   if (l!=NULL) {
      ret->prev=l->last;
   } else {
      ret->prev=NULL;
   }
   if (l!=NULL && l->last!=NULL) {
      l->last->next=ret;
   }
   ret->next=NULL;
   ret->id=strdup(id);
   ret->data=data;

   if (l==NULL) return ret;

   if (l->first==NULL) l->first=ret;
   l->last=ret;

   return ret;
}

Эх, проверяй, не проверяй, один хрен все грохнется при l==NULL

Pythoner, 21 Ноября 2014

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

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

+136

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

len += sprintf(event_xml_msg, XML_TAG_START, XML_KOKOKO_HTTP_PROTOCOL);

	// Set <monitor-event>
	len += sprintf(strend_ptr(event_xml_msg), XML_TAG_START, XML_MONITOR_EVENT_NODE_TREE);

	// Set <date>
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_DATE, dt.date_b);

	// Set <time>
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_TIME, dt.time_b);

	// Set <product> Ex. "VersAtive"
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_PRODUCT, product_type);

	// Set <entity code>
	// Supposed to work for all union types
	len += xml_int_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_CODE, event_code);

	// Set <severity>
	//	len += xml_int_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_SEVERITY, severity);
	memset(severity_str, 0, sizeof(severity_str));
	get_severity_string(severity, severity_str);
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_SEVERITY, severity_str);

	// Set event entity name
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_ENTITY_TYPE, entity_name);

	// Set event description
	if((len + strlen(description)) > (payload_size - footer_size))
	{
		// TODO HANDLE
		printf("Message description overflows buffer size.\n");
		return false;
	}
	len += xml_cdata_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_DESCRIPTION, description);

	// Set params
	add_xml_entity_params(event_xml_msg, entity_params);

	// Close <monitor-event>
	sprintf(strend_ptr(event_xml_msg), XML_TAG_END, XML_MONITOR_EVENT_NODE_TREE);

	// Close <HTTPProtocol>
	len += sprintf(strend_ptr(event_xml_msg), XML_TAG_END, XML_KOKOKO_HTTP_PROTOCOL);

В проекте широко используется libmxml, а вот блять использовать его по назначению велосипедики не могут.

codemonkey, 20 Ноября 2014

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

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

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

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

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

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

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

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

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

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

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

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