SOS. Ошибка при запонении списка из процедуры обработки прерывания

[Alice1144]

Подскажите в чем моя ошибка и откуда берется ошибка "Null pointer assignment".
Программа по прерыванию от таймера заполняет список, в основном модуле читает список и выводит на экран. Выводит:
 10 10 10 10 10 10 10 10 10 10 Null pointer assignment
Должно быть:
 1 2 3 4 5 6 7 8 9 10
Текст программы:

Код:

//#include <stdio.h>
#include <dos.h>
#include <conio.h>
#include <iostream.h>
//#include <alloc.h>
#include <stdlib.h>
//#include <bios.h>;
//#include <process.h>

#ifdef __cplusplus
    #define __CPPARGS ...
#else
    #define __CPPARGS
#endif

struct BuffList{
	unsigned int VK;
	struct BuffList *next;
} start, *node, *first, *last;

int looping = 1;
void interrupt get_out(__CPPARGS);
void interrupt (*oldfunc)(__CPPARGS);

void main(void)
{ clrscr();
  start.next = NULL;
  node = first = last= &start;
  oldfunc  = _dos_getvect(0x1c);
  _disable();
  _dos_setvect(0x1c,get_out);
  _enable();
  sleep(2); //ждем пока заполнится список
  first=first->next;
  for (char k=1;k<11;k++)
  {
    node=first;
    cout<<node->VK<<"\r\n";
    first=node->next;
    //free(node);
  }
  _disable();
  _dos_setvect(0x1c,oldfunc);
  _enable();
}

void interrupt get_out(__CPPARGS)  //прерывание от часов 18 раз в секунду
{ if (looping>10) return;
  last->next =(struct BuffList*)malloc(sizeof(struct BuffList));
  last=last->next;
  last->VK=looping++;
  last->next=NULL;
}

[Алексей++]

насчёт
Null pointer assignment

наверное, выполняется нечто вроде этого:

const char* p=0;
cout<<p;

[Алексей++]

цикл

Код:

  for (char k=1;k<11;k++)
  {
    node=first;
    cout<<node->VK<<"\r\n";
    first=node->next;
    //free(node);
  }

насчёт всех десяток - значит действительно там они. Ошибка в другом месте. А тут ошибка такая, что нет проверки на 0 указателя, вот так красивее:

Код:

  first=first->next;
  for(;first;)
  {
    node=first;
    cout<<node->VK<<"\r\n";
    first=node->next;
  }

[Alice1144]

насчёт
Null pointer assignment

ошибка вылазит после окончания всех команд в main на выходе из программы.

Насчет десяток. Проанализировав в отладчике вижу, что создается первый элемент списка и он же ссылается на самого себя, в нем меняеется только значение VK (сохраняется последняя 10). Хотя в обработке прерывания

void interrupt get_out(__CPPARGS)  //прерывание от часов 18 раз в секунду
{ if (looping>10) return;
  last->next =(struct BuffList*)malloc(sizeof(struct BuffList));
  last=last->next;
  last->VK=looping++;
  last->next=NULL;
}

прописано что каждый элемент списка ссылается на следуюший. Как устранить ошибку непонятно.
Из-за этого неправильно и считываются данные

[Алексей++]

это МК ? Или компьютер ? Если комп, то не забывай также освобождать выделенную память, а то будет утечка.

Как исправить ошибку. Сначала надо узнать, что должно получиться в итоге

предполагаю, что

Код:

void interrupt get_out(__CPPARGS)  //прерывание от часов 18 раз в секунду
{
  if (looping>10) return;
  struct BuffList* a_new =(struct BuffList*)malloc(sizeof(struct BuffList));

  if(!first)
  {
    first=a_new;
  }

  last=a_new;
  last->VK=looping++;
  last->next=NULL;
}

...

Код:

//распечатка
  struct BuffList* curr=first;
  for(;curr;)
  {
    cout<<curr->VK<<"\r\n";
    curr=curr->next;
  }

[Alice1144]

это МК ? Или компьютер ?
Сначала надо узнать, что должно получиться в итоге

Это комп c TurboC++3.
В итоге должно быть 1 2 3 4 5 6 7 8 9 10
В более общем случае будет прерывание от LPT (с частотой несколько кГц), чтение из него, запись в BuffList, а параллельно чтение из BuffList и обработка данных

Проверил Ваши предложения. Ваша обработка прерывания дает те же результаты.
А распечатка

Код:

//распечатка
  struct BuffList* curr=first;
  for(;curr;){}

вылетает не входя в цикл.

Вставлял текст из обработки прерывания в main (заполнял циклом) все работает.
Предполагаю, что почему-то неправильно работает malloc при работе с прерыванием.
Жду еще предложений.

[Алексей++]

хм...
1) проверять надо не мои предположения, а свою логику
2) неинициализирован first.

Код:

void main(void)
{
  clrscr();
  //инициализация
  first = 0;

попробуй вот такую структуру программы

Код:

//#include <stdio.h>
#include <dos.h>
#include <conio.h>
#include <iostream.h>
//#include <alloc.h>
#include <stdlib.h>
//#include <bios.h>;
//#include <process.h>

#ifdef __cplusplus
#define __CPPARGS ...
#else
#define __CPPARGS
#endif

struct BuffList
{
	unsigned int VK;
	BuffList* next;

	BuffList()
	{
		VK=0;
		next=0;
	}
};

struct s_array
{
	BuffList* first;//первый элемент списка	
	BuffList** new_next;//указатель на последний, пустой элемент списка
	int looping;

	s_array():
	{
		first=0;
		new_next=0;
		looping=1;
	}

	~s_array()
	{
		new_next=0;

		//чистим память
		BuffList* nextToDel=0;
		BuffList* currToDel=first;
		for(;currToDel;)
		{
			nextToDel=currToDel->next;
			free currToDel;
		}
	}

	BuffList* AddNew()
	{
		if (looping>10) return 0;

		if(!new_next)
		{
			new_next=&first;
		}

		*new_next=(struct BuffList*)malloc(sizeof(struct BuffList));

		BuffList* result=*new_next;
		new_next=&new_next->next;
		
		result->VK=looping++;
		return result;
	}

	//распечатка
	void Print() const
	{
		BuffList* curr=first;
		for(;curr;)
		{
			cout<<curr->VK<<"\r\n";
			curr=curr->next;
		}
	}
};

s_array MyArray;


void interrupt get_out(__CPPARGS);
void interrupt (*oldfunc)(__CPPARGS);

void main(void)
{
	clrscr();

	oldfunc=_dos_getvect(0x1c);

	_disable();
	_dos_setvect(0x1c,get_out);
	_enable();

	//ждем пока заполнится список
	sleep(1000);

	//распечатка
	MyArray.Print();

	_disable();
	_dos_setvect(0x1c,oldfunc);
	_enable();
}

void interrupt get_out(__CPPARGS)  //прерывание от часов 18 раз в секунду
{
	MyArray.AddNew();
}

(не тестировал)

[Sla]

хм, использовать heap  прерываниях не есть хорошо, даже если в однозадачной ОС. Передай в прерывание статический массив.

[Alice1144]

Вы писали

Код:

	BuffList* AddNew()
	{
		if (looping>10) return 0;

		if(!new_next)
		{
			new_next=&first;
		}

		*new_next=(struct BuffList*)malloc(sizeof(struct BuffList));

		BuffList* result=*new_next;
		new_next=&new_next->next;
		
		result->VK=looping++;
		return result;
	}

Извините, что пропал надолго: сдох интернет.
Проверить Вашу версию проги не получается, вылазит ошибка на строке:
   new_next=&new_next->next;
И что-то мне не удается ее обойти.

Но зато заработала моя первая версия проги:
заменил sleep(1) на delay(1000) и после этого получил ожидаемый результат.
Почему sleep(1) глючит не очень понятно, может он тоже использует это прерывание и конфликтует.

Спасибо за содействие.

[Алексей++]

а, ну да,

new_next=new_next->next;

[Рыжий Тигра]

Alice1144, malloc() изнутри таймерного прерывания - оригинально. Уверен, что при таком подходе никогда не прервёшь DOS'овую функцию (ту же sleep())? А ведь DOS ни разу не реентерабельный! Выделяй массив заранее, до _dos_setvect()'а!

[Dimka]

Рыжий Тигра, не всё так уж страшно, если: 1) из под перегрузившего прерывание процесса не запускаются другие, 2) процесс не являтся TSR.

Проблема может быть только в том, что внутри malloc нет средств синхронизации основного потока команд, и потока команд, возникающего при срабатывании прерывания.

Тогда надо написать свою функцию-обёртку malloc, которая гарантирует последовательное срабатывание malloc в двух потоках. В простейшем случае функциями enable и disable блокировки прерываний из dos.h.