Форум программистов, компьютерный форум CyberForum.ru
Наши страницы

C++

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 10, средняя оценка - 4.90
-=ЮрА=-
Заблокирован
Автор FAQ
#1

Помогите выловить баг, сил моих больше нету((( - C++

06.04.2011, 11:06. Просмотров 1261. Ответов 15
Метки нет (Все метки)

Программа глючит при добавлении больших структурированных каталогов, раньше думал, что ошибка выделения памяти, оказалось нет...Очень надо помощь спеца, СРОЧНО!!!
Конструктор класса
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
CStorage::CStorage()
{
    CoInitialize(NULL);
    GetTempPath(MAX_PATH,szTMP);
    GetShortPathName(szTMP,szTMP,MAX_PATH);
    lpszW = (WCHAR *)malloc(sizeof(WCHAR));
    lpBuf = malloc(sizeof(char));
    //MEM_TOP_DOWN
/*  pDATA = (WIN32_FIND_DATA *)VirtualAlloc
                        (
                            (void *)&pDATA,
                            sizeof(WIN32_FIND_DATA),
                            MEM_COMMIT  ,
                            PAGE_READWRITE
                        );*/
    pDATA = (WIN32_FIND_DATA *)malloc(sizeof(WIN32_FIND_DATA));
    pFile = NULL;
    m_hWnd= NULL;
    nStgElm= 0;
    uStgLen= 0;
    pDATA[nStgElm].dwReserved1 = 0;
    iINDENT = 0;
}
Рекурсивный поиск каталогов, ошибка вылетает в try - catch блоке
C++
1
2
3
4
5
6
7
8
9
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
73
WIN32_FIND_DATA * CStorage::MkPathList(LPCTSTR szPath)
{
    WIN32_FIND_DATA pFILE_DATA;ULONG uIND = nStgElm; 
    pDATA[nStgElm].dwFileAttributes = GetFileAttributes(szPath);
    pDATA[nStgElm].dwReserved1      = iINDENT;
    HANDLE hFile = CreateFile(
        strcpy(pDATA[nStgElm].cFileName,szPath), 
            GENERIC_READ,
            FILE_SHARE_READ,
            NULL, 
            OPEN_EXISTING, 
            FILE_ATTRIBUTE_NORMAL,
            NULL);
    if(hFile != INVALID_HANDLE_VALUE)
    {
        uStgLen += (pDATA[nStgElm].nFileSizeLow = GetFileSize(hFile,NULL));
        CloseHandle(hFile);
    }
    else
    {
        if(strcmp(pDATA[nStgElm].cFileName,"") == 0)
           strcpy(pDATA[nStgElm].cFileName,"+");
        sprintf(pFILE_DATA.cFileName,"%s\\*.*",pDATA[nStgElm].cFileName);
        hFile = FindFirstFile(pFILE_DATA.cFileName,&pFILE_DATA);
        if(hFile != INVALID_HANDLE_VALUE)
        {
            iINDENT++;
            pDATA[(uIND = nStgElm)].nFileSizeLow = uStgLen;
            pDATA = (WIN32_FIND_DATA *)realloc
                        (
                            (void *)pDATA,
                            sizeof(WIN32_FIND_DATA)*(1 + (nStgElm = nStgElm + 1))
                        );
            do
            {
                if(strcmp(pFILE_DATA.cFileName,".")  == 0 ||
                   strcmp(pFILE_DATA.cFileName,"..") == 0)
                    continue;
                else
                {
                    pFILE_DATA.dwReserved1 = iINDENT;
                    pDATA[nStgElm] = pFILE_DATA;
                    sprintf(pDATA[nStgElm].cFileName,"%s\\%s",szPath,pFILE_DATA.cFileName);
                    if(FILE_ATTRIBUTE_DIRECTORY & pFILE_DATA.dwFileAttributes)
                    try
                    {
                        MkPathList(pDATA[nStgElm].cFileName);
                    }
                    catch(...)
                    {
                        iINDENT--;
                        //MessageBox(m_hWnd,strerror(GetLastError()),"ERROR",MB_OK);
                    }
                    else
                    {
                        uStgLen += pFILE_DATA.nFileSizeLow;
                        pDATA = (WIN32_FIND_DATA *)realloc
                                    (
                                        (void *)pDATA,
                                        sizeof(WIN32_FIND_DATA)*(2 + (nStgElm = nStgElm + 1))
                                    );
                    }
                    PumpMessages(m_hWnd);
                }
            }
            while(FindNextFile(hFile,&pFILE_DATA) != 0);
            FindClose(hFile);
            pDATA[uIND].nFileSizeLow = uStgLen - pDATA[uIND].nFileSizeLow;
            iINDENT--;
        }
    }
    return pDATA;
}
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
06.04.2011, 11:06
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Помогите выловить баг, сил моих больше нету((( (C++):

Баг asio? или баг TCP стека? - C++
всем привет. повстречался с очень странным багом. и не могу определить кто бажит, asio, или TCP-стек. на стороне клиента,...

С++Builder сил моих не хватает - C++ Builder
Задача. В одномерном массиве, состоящем из 14 вещественных элементов, вычислить: 1) минимальный элемент массива; 2) сумму элементов,...

Как выловить, что вызывает Exception? - C++ Builder
Программа принимает данные через DDE и пишет их в StringGrid. Может часами висеть в свернутом виде и нормально работать (по таймеру в 5...

помогите определить баг или недочёт в программе. - C++
Задача ввести с клавиатуры массив и вывести на экран наибольший его элемент. Проблема заключается в if когда s<mas он всё время...

std::regex : баг на сайте или баг компилятора? - C++
http://en.cppreference.com/w/cpp/regex/regex_match этот код выкидывает throw... Добавлено через 35 секунд компилятор gcc 4.8

Перевод из си в ассемблер.Сил моих больше нет. - Assembler
Вообщем,задача состоит в следующем перевести сишный код в ассемблерный. Часть кода есть,но т.к. писался он в ускоренном темпе,то есть...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
taras atavin
Ушёл с форума.
3569 / 1753 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 07:54 #2
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
ошибка вылетает в try - catch блоке
Ты б его хоть показал, а то телепат всего то один, да и тот на форуме игроделов.
0
ForEveR
В астрале
Эксперт С++
7973 / 4735 / 321
Регистрация: 24.06.2010
Сообщений: 10,542
Завершенные тесты: 3
11.04.2011, 09:06 #3
taras atavin, 45-52 строчка во втором листинге
0
taras atavin
Ушёл с форума.
3569 / 1753 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:24 #4
Вылетает конкретно на 51-й? Если кач вообще сработал, то ищи, откуда выплыло само исключение. И iINDENT где то объявлен и мне просто опять не по глазам? Какой у него тип?
0
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 09:39  [ТС] #5
taras atavin, программа вылетает при рекурсивном вызове функции MkPathList и то при большой вложенности подкаталогов, это сначало наталкнуло меня на мысль что realloc не справляется с выделением памяти, но если бы происходили косяки с выделением памяти блок-памяти отведенный под pDATA обнулялся, а так с трай кэтч прога работает дальше. Я специально не хочу вводить в СStorage лист-контрол куда и осуществляю выгрузку результатов поиска, этот класс хочу юзать как для консолей так и диалоговых приложений. Параметр iINDENT отвечает за сдвиг иконок в листбоксе, т.е. указывает на вложенность подкаталога, вот привожу скриншот работы проги
0
Миниатюры
Помогите выловить баг, сил моих больше нету(((  
taras atavin
Ушёл с форума.
3569 / 1753 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:44 #6
Так с отловленным исключением всё работает? Тогда сделай лог и вывод максимально расширенного сообщения об исключении и ищи, откуда оно лезет.
0
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 09:46  [ТС] #7
PS: iINDENT = 0; - 22-ая строчка в конструкторе, в объяве мог не показать, вырезал класс с проги
ну и сокращал
0
taras atavin
Ушёл с форума.
3569 / 1753 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:47 #8
Каков полный путь к тому файл/подкаталогу, на котором выскакивает исключение? Какой он длины, сколько в нём уровней? Допустимо ли это с точки зрения как системы в целом, так и конкретно используемых функций.
1
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 09:49  [ТС] #9
Та пишет i/o error и всё - мне этого мало чтобы выявить баг. Ввёл рапорт об ошибке, но тоже мало чем помогло(((

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
CString CStgMakerDlg::StrWinError(HRESULT hRes)
{
    CString str;
    str.Format("\r\nHRESULT : 0x%p\r\n(int)hRes : %d\r\n",hRes,hRes);
    str+="ERRMSG  : ";
    LPSTR errString = NULL;  // will be allocated and filled by FormatMessage
    FormatMessage(
        FORMAT_MESSAGE_ALLOCATE_BUFFER |
        FORMAT_MESSAGE_FROM_SYSTEM, // use windows internal message table
        0,       // 0 since source is internal message table
        hRes,    // this is the error code 
                          // Could just as well have been an error code from generic
                          // Windows errors from GetLastError()
        0,       // auto-determine language to use
        (LPSTR)&errString,
        0,       // min size for buffer
        0 ); 
    return str += errString;
}
0
taras atavin
Ушёл с форума.
3569 / 1753 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:50 #10
Может быть ты просто влез в ограничение?

Добавлено через 55 секунд
А i/o егор - это ошибка обращения к диску.
0
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 17:22  [ТС] #11
taras atavin я перепаял прогу вобщем прога вылетает в try_catch который приведен в коде ниже, причём для каждого каталог в одном конкретном месте, в некоторых каталогах вообще исключения нет,
strerror пишет Improper link. Моё предположение что FindFirstFile обнуляет pFILE_DATA и вот здесь
C++
1
pDATA[nStgElm] = pFILE_DATA;
мы делаем
C++
1
 pDATA[nStgElm] = NULL
Помоги как можно исключение обрабоать???
C++
1
2
3
4
5
6
7
8
9
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
do
            {
                if(strcmp(pFILE_DATA.cFileName,".")  == 0 ||
                   strcmp(pFILE_DATA.cFileName,"..") == 0)
                    continue;
                else
                {
                    pFILE_DATA.dwReserved1 = iINDENT;
                    try
                    {
                    pDATA[nStgElm] = pFILE_DATA;
                    sprintf(pDATA[nStgElm].cFileName,"%s\\%s",szPath,pFILE_DATA.cFileName);
                    }
                    catch(...)
                    {
                        sprintf
                            (
                                chERR,
                                "do-while: begin\r\n%s\r\n%s\r\n%s\r\n%s",
                                strerror(GetLastError()),
                                pDATA[nStgElm - 2].cFileName,
                                pDATA[nStgElm - 1].cFileName,
                                pFILE_DATA.cFileName
                            );
                        MessageBox(m_hWnd,chERR,strerror(GetLastError()),MB_OK);
                    }
                    if(FILE_ATTRIBUTE_DIRECTORY & pFILE_DATA.dwFileAttributes)
                        MkPathList(pDATA[nStgElm].cFileName);
                    else
                    {
                        uStgLen += pFILE_DATA.nFileSizeLow;
                        pDATA = (WIN32_FIND_DATA *)realloc
                                    (
                                        (void *)pDATA,
                                        sizeof(WIN32_FIND_DATA)*(1 + (nStgElm = nStgElm + 1))
                                    );
                    }
                    PumpMessages(m_hWnd);
                }
            }
            while(FindNextFile(hFile,&pFILE_DATA) != 0);
0
taras atavin
Ушёл с форума.
3569 / 1753 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
12.04.2011, 08:22 #12
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
обнуляет pFILE_DATA
То есть файл просто не найден? В сообщении об исключении выведи значение pFILE_DATA для проверки, потом сделай проверку ифом.
0
ValeryS
Модератор
6634 / 5041 / 466
Регистрация: 14.02.2011
Сообщений: 16,854
12.04.2011, 08:38 #13
Может срабатывает ограничение на длину имени файла???
макрос есть MAXPACH по моему.
определен в 255
попробуй добавить.
где то писал рекурсивный поиск файлов
найду исходник вышлю

Добавлено через 2 минуты
GetLastError() что пишет
0
-=ЮрА=-
Заблокирован
Автор FAQ
12.04.2011, 09:47  [ТС] #14
Баг уже выловил - как оказалось при определённой структуре внутренних подкаталогов добавляемой дирректории, происходит перекрытие блока памяти в котором содержится LPCTSTR szPath, в резулльтате szPath == NULL, об этом говорит указанный скриншот, проблемму решил введением дополнительной строковой переменной для хранения пути к каталогу, все проблеммы исчезли!
Ребят спасибо всем за участие!
0
Миниатюры
Помогите выловить баг, сил моих больше нету(((  
ValeryS
Модератор
6634 / 5041 / 466
Регистрация: 14.02.2011
Сообщений: 16,854
12.04.2011, 19:40 #15
C++
1
2
3
4
5
6
7
8
9
10
11
12
typedef struct _WIN32_FIND_DATA {  
DWORD dwFileAttributes; 
 FILETIME ftCreationTime; 
 FILETIME ftLastAccessTime; 
 FILETIME ftLastWriteTime; 
 DWORD nFileSizeHigh;
  DWORD nFileSizeLow; 
 DWORD dwReserved0; 
 DWORD dwReserved1;
 TCHAR cFileName[MAX_PATH]; 
TCHAR cAlternateFileName[14];
} WIN32_FIND_DATA
TCHAR cFileName[MAX_PATH]; !!!!
WinDef.h
C++
1
#define MAX_PATH          260
о чем я и писал
преопредели MAX_PATH
сделай 1024 и ошибка должна исчезнуть
хотя наверно можно найти и более длинное имя файла
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
12.04.2011, 19:40
Привет! Вот еще темы с ответами:

Сложный запрос, нету сил больше - MySQL
Есть таблица в которой много полей, выбрать нужно все. Есть 2 поля с помощью которых нужно сформировать ответ - code (название,...

Сил моих нет Денвер постоянно ругается на рабочие коды - PHP
Warning: session_start() : Cannot send session cookie - headers already sent by (output started at...

Сил больше нет! - Игровой компьютер
Всем привет! Комп - мать asus p8p67 rex1.xx, i5, gtx650, 8gb оперативки Проблема следующая: 1. Высокая температура ЦП - почистил...

Виснет компьютер, сил больше нет - Компьютерное железо
Здравствуйте, видел уже много подобных ситуаций, но ни один из советов мою проблему не решил. Начну с начала, купил компьютер.Windows XP...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
12.04.2011, 19:40
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru