graywolf's lair » Code WTFs

Code WTF #3

graywolf — Thu, 18 Dec 2008 15:09:11 +0000

Сьогодні ще цікавого надибав:

bool test = (mail.Find('@') == -1);
if(tsInspireDate.GetDays()<15 && !active && (mail.Find('@') == -1))
{ /* ...skipped... */ }

І ніби ж не індуси…

P.S. Хоча, звісно, всяке буває… Я і у себе в коді, що часто перероблювався, якось перли знаходив (коли один дрібний фрагмент змінюєш, а попередній шматок коду, що його дублює забуваєш видалити).

Driver code WTF

graywolf — Thu, 25 Sep 2008 07:48:09 +0000

Вчора наш драйверщик шукаючи одну багу в коді драйвера знайшов іншу, непримітну, залишену колишнім розробником цього драйвера. Знайдіть її і ви. Це гарна вправа на знання деяких хитрих програмістських методик.

Сирці я трохи підравив, повикидавши згадки назви драйверу та більшу частину несуттєвого до баги коду і додавши своє визначення SecurityCookie (це, до речі, теж підказка ).

ULONG SecurityCookie = 0xDEADBEEF;

[ ... skipped ... ]

pUserNotificationMap->MappedSize =
    FloatingUnitSize
    + sizeof(NOTIFICATION_MAP)
    + sizeof(SecurityCookie);

pUserNotificationMap->MappedUnits = 1;

RtlCopyBytes(
    &pUserNotificationMap->NotificationUnit[0].NotificationInfo,
    &NotifInfo,
    sizeof(NotifInfo) );

*((PULONG)((PCHAR)pUserNotificationMap+pUserNotificationMap->MappedSize)) = SecurityCookie;

status = CommitNotificationUnit(
    pContext,
    pInternalPolicyRule,
    pCtxString,
    pCtxThread,
    &pUserNotificationMap->NotificationUnit[0],
    sizeof(NOTIFICATION_UNIT) + FloatingUnitSize );

//
// Overflowed? Check out!
//
if( *((PULONG)((PCHAR)pUserNotificationMap+pUserNotificationMap->MappedSize))
        != SecurityCookie )
{
    status = STATUS_HEAP_CORRUPTION;
    ReportDesignError_Leave( "Memory corruption detected!" );
}

P.S. Підказка: “за що боролись, на те й напоролись”

Code WTF

graywolf — Thu, 10 Jul 2008 10:19:05 +0000

Вчора потрібно було пофіксити багу в продукті і поки придивлявся місце для фіксу натрапив на таке:

 ATL::CString strText;

 SYSTEMTIME st = {0};
 GetLocalTime(&st);

 if (st.wHour<10)
     strText.Format(_T("%d/%d/%d, 0%d:%d"),
         st.wDay,st.wMonth,st.wYear, st.wHour, st.wMinute);
 else
     strText.Format(_T("%d/%d/%d, %d:%d"),
         st.wDay,st.wMonth ,st.wYear, st.wHour, st.wMinute);

 m_date.SetWindowText(strText);

Ні, код абсолютно робочий, але ж…

m_date.SetWindowText(CTime::GetCurrentTime().Format(_T("%d/%m/%Y %H:%M")));

По-моєму, краще, нє?

P.S. До речі, мабуть заведу собі в блозі відповідний розділ з wtf’ами