QT segmentation falut when communicating with a bluetooth function..

[medampudi]

Hi every one,

I am trying to integrate the the bluetooth functionality with the QT applciation. I was just wondering what would be the problem with the sprintf statement(tested out both) but does not work...

the function is called from the main function which specifies the struct and initialises its comPort ... Then the data are passed to the connectRobot function and hence this error...

A error of segmentation is shown up if i debug the program at line 9.... crashing the whole program when I run it...

any suggestions on the mistake i would have made...


Thanking you...

Rajesh Medampudi

int connectRobot(struct robot *rbt, int comPort)
{
 
 
 
	char portname[20];
        sprintf(portname,"COM%d",comPort);
        //sprintf(portname,"\\\\.\\COM%d",comPort);
	hSerialPort[rbt->serialConnection] = CreateFileA(portname,
						GENERIC_READ | GENERIC_WRITE,
						0,
						0,
						OPEN_EXISTING,
						FILE_ATTRIBUTE_NORMAL,
						0);
	if (hSerialPort[rbt->serialConnection] == INVALID_HANDLE_VALUE) return -1;
 
 
 
	dcbSerialParams.DCBlength=sizeof(dcbSerialParams);
	if (!GetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams)) return -2;
 
	dcbSerialParams.BaudRate = 115200;
	dcbSerialParams.ByteSize = 8;
	dcbSerialParams.StopBits = ONESTOPBIT;
	dcbSerialParams.Parity	 = NOPARITY;
 
	comt.ReadIntervalTimeout = 500;
	SetCommTimeouts( hSerialPort[rbt->serialConnection], &comt);
	if (!SetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams)) return -3;
 
	rbt->port=comPort;
	rbt->connected=1;
 
 
 
	return 0;
}

int connectRobot(struct robot *rbt, int comPort)
{



	char portname[20];
        sprintf(portname,"COM%d",comPort);
        //sprintf(portname,"\\\\.\\COM%d",comPort);
	hSerialPort[rbt->serialConnection] = CreateFileA(portname,
						GENERIC_READ | GENERIC_WRITE,
						0,
						0,
						OPEN_EXISTING,
						FILE_ATTRIBUTE_NORMAL,
						0);
	if (hSerialPort[rbt->serialConnection] == INVALID_HANDLE_VALUE) return -1;



	dcbSerialParams.DCBlength=sizeof(dcbSerialParams);
	if (!GetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams)) return -2;

	dcbSerialParams.BaudRate = 115200;
	dcbSerialParams.ByteSize = 8;
	dcbSerialParams.StopBits = ONESTOPBIT;
	dcbSerialParams.Parity	 = NOPARITY;

	comt.ReadIntervalTimeout = 500;
	SetCommTimeouts( hSerialPort[rbt->serialConnection], &comt);
	if (!SetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams)) return -3;

	rbt->port=comPort;
	rbt->connected=1;



	return 0;
}

To copy to clipboard, switch view to plain text mode 

[wysota]

Where do you initialize hSerialPort and how do you call connectRobot()?

[medampudi]

I just initialized them right before in the same file at the start....

struct robot robots[7];
 
 
	HANDLE		 hSerialPort[7];  //up to 7 connection
	DCB			 dcbSerialParams={0};
	COMMTIMEOUTS comt;
 
// fine
 
 
 
//stat
clock_t startTime;
clock_t endTime;
 
int dtime;

struct robot robots[7];


	HANDLE		 hSerialPort[7];  //up to 7 connection
	DCB			 dcbSerialParams={0};
	COMMTIMEOUTS comt;

// fine



//stat
clock_t startTime;
clock_t endTime;

int dtime;

To copy to clipboard, switch view to plain text mode 

and connect robot is called something like this....

connectRobot(&robots[1], 1036);

connectRobot(&robots[1], 1036);

To copy to clipboard, switch view to plain text mode 

[wysota]

And what does the constructor for struct robot look like? Is the serialConnection field initialized properly?

[medampudi]

anything else that i need for this....

struct robot
{
	int connected;
	int serialConnection; //refers to HANDLE vector hSerialPort
	int port;
 
	//Sensors
	int IR[8]; //infrared sensors 'N'
	int LI[8]; //light sensors    'O'
	int FL[3]; //floor sensors    'M'
	int AC[3]; //acceleration s.  'A'
	int MC[3]; //microphone       'U'
	int MR[6]; //microretina //cam is dived in 6 sector an then for each sector the number of pixel of a target color is computed
	char videoBuffer[1600]; //img 'I'
	int imageRGB[1600];     //rgb image built upon videobuffer
	char sensorsBuffer[60];
	int camW; //cam width         'X cam_mode,cam_widht, cam_heigth,cam_zoom :expressed in byte
	int camH; //cam height
	int camZ; //cam zoom
	//used by  Joachim
	int old_floor_input[2]; //floor input from previous timestep
	int	floor_mean[3];		//mean floor value from previous timesteps
	int target_prototype[3];//prototypes of floorsensor values for [0]=no target(white), [1]=target1(grey), [2]=target2(black)
	int	epuck_angle_old;	//angle of other epuck at previous timestep
	float old_input[3];		//camera input from previous timestep
 
 
};

struct robot
{
	int connected;
	int serialConnection; //refers to HANDLE vector hSerialPort
	int port;

	//Sensors
	int IR[8]; //infrared sensors 'N'
	int LI[8]; //light sensors    'O'
	int FL[3]; //floor sensors    'M'
	int AC[3]; //acceleration s.  'A'
	int MC[3]; //microphone       'U'
	int MR[6]; //microretina //cam is dived in 6 sector an then for each sector the number of pixel of a target color is computed
	char videoBuffer[1600]; //img 'I'
	int imageRGB[1600];     //rgb image built upon videobuffer
	char sensorsBuffer[60];
	int camW; //cam width         'X cam_mode,cam_widht, cam_heigth,cam_zoom :expressed in byte
	int camH; //cam height
	int camZ; //cam zoom
	//used by  Joachim
	int old_floor_input[2]; //floor input from previous timestep
	int	floor_mean[3];		//mean floor value from previous timesteps
	int target_prototype[3];//prototypes of floorsensor values for [0]=no target(white), [1]=target1(grey), [2]=target2(black)
	int	epuck_angle_old;	//angle of other epuck at previous timestep
	float old_input[3];		//camera input from previous timestep


};

To copy to clipboard, switch view to plain text mode 

[wysota]

But do you initialize the object in any way? If not, then rbt->serialConnection will return a random value and you will dereference the hSerialPort array in a wrong way causing a crash of your program.

[medampudi]

Is this what was on your mind....

int initRobots()
{
	int id;
	for(id=0;id<7;id++)
	{
		robots[id].serialConnection=id;
		robots[id].port=-1;
		robots[id].connected=0;
	}
return 0;
}

int initRobots()
{
	int id;
	for(id=0;id<7;id++)
	{
		robots[id].serialConnection=id;
		robots[id].port=-1;
		robots[id].connected=0;
	}
return 0;
}

To copy to clipboard, switch view to plain text mode 

[wysota]

Where do you call it? By the way, I suggest you make your code less C-ish and more C++-ish, it will get much more cleaner and more readable.

[medampudi]

Thank you for the suggestion and i have tried to change a lot of things in my code. i would also be adding the codes and the errors...

the cpp file...

#include "epucksercomm.h"
 
 
epuckSercomm::epuckSercomm()
{
 
}
int epuckSercomm::initRobots(struct robot *rbt)
{
        int id;
        for(id=0;id<7;id++)
        {
                rbt[id].serialConnection=id;
                rbt[id].port=-1;
                rbt[id].connected=0;
        }
return 0;
}
int epuckSercomm::connectRobot(struct robot *rbt, int comPort)
{
 
 
 
        char portname[20];
        //sprintf(portname,"COM%d",comPort);
        sprintf(portname,"\\\\.\\COM%d",comPort);
        hSerialPort[rbt->serialConnection] = CreateFileA(portname,
                                                GENERIC_READ | GENERIC_WRITE,
                                                0,
                                                0,
                                                OPEN_EXISTING,
                                                FILE_ATTRIBUTE_NORMAL,
                                                0);
        if (hSerialPort[rbt->serialConnection] == INVALID_HANDLE_VALUE) return -1;
 
 
 
        dcbSerialParams.dcb.DCBlength=sizeof(dcbSerialParams);
        if (!GetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams.dcb)) return -2;
 
        dcbSerialParams.dcb.BaudRate = 115200;
        dcbSerialParams.dcb.ByteSize = 8;
        dcbSerialParams.dcb.StopBits = ONESTOPBIT;
        dcbSerialParams.dcb.Parity     = NOPARITY;
 
        comt.ReadIntervalTimeout = 500;
        SetCommTimeouts( hSerialPort[rbt->serialConnection], &comt);
        if (!SetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams.dcb)) return -3;
 
        rbt->port=comPort;
        rbt->connected=1;
 
 
 
        return 0;
}
 
 
int epuckSercomm::setRobotSpeed(struct robot *rbt, int speedL, int speedR)
{
 
 
 
        DWORD dwBytesReadWrite = 0;
        char command[5];
 
        command[0]='D';
    command[1]=speedL & 0xff;
    command[2]=speedL >> 8;
    command[3]=speedR & 0xff;
    command[4]=speedR >>8;
 
        if (!WriteFile(hSerialPort[rbt->serialConnection],command,5,&dwBytesReadWrite,NULL)) return -1;
 
 
 
        return 0;
 
 
}
int epuckSercomm::disconnectRobot(struct robot *rbt)
{
 
        CloseHandle(hSerialPort[rbt->serialConnection]);
        rbt->connected=0;
 
 
return 0;
}
int epuckSercomm::printRobotSensors(struct robot *rbt)
{
        printf("Robot no:%d\r\n",rbt->serialConnection);
 
        printf("IR: %d %d %d %d %d %d %d %d\r\n",
                rbt->IR[0],rbt->IR[1],rbt->IR[2],rbt->IR[3],
                rbt->IR[4],rbt->IR[5],rbt->IR[6],rbt->IR[7]);
 
        printf("LI: %d %d %d %d %d %d %d %d\r\n",
                rbt->LI[0],rbt->LI[1],rbt->LI[2],rbt->LI[3],
                rbt->LI[4],rbt->LI[5],rbt->LI[6],rbt->LI[7]);
 
        printf("FL: %d %d %d\r\n",
                rbt->FL[0],rbt->FL[1],rbt->FL[2]);
 
        printf("MC: %d %d %d\r\n",
                rbt->MC[0],rbt->MC[1],rbt->MC[2]);
 
return 0;
 
}
int  epuckSercomm::setRobotLed(struct robot *rbt,int noLed, int value)
{
 
 
        DWORD dwBytesReadWrite = 0;
        char command[3];
        if (value<0)value=0; //on
        if (value>1)value=1; //off
 
        command[0]='L';
    command[1]= noLed;   //number of led to switch
    command[2]= value;  //0: off, 1: on
 
        if (!WriteFile(hSerialPort[rbt->serialConnection],command,3,&dwBytesReadWrite,NULL))
                return -1;
 
 
        return 0;
 
 
}

#include "epucksercomm.h"


epuckSercomm::epuckSercomm()
{

}
int epuckSercomm::initRobots(struct robot *rbt)
{
        int id;
        for(id=0;id<7;id++)
        {
                rbt[id].serialConnection=id;
                rbt[id].port=-1;
                rbt[id].connected=0;
        }
return 0;
}
int epuckSercomm::connectRobot(struct robot *rbt, int comPort)
{



        char portname[20];
        //sprintf(portname,"COM%d",comPort);
        sprintf(portname,"\\\\.\\COM%d",comPort);
        hSerialPort[rbt->serialConnection] = CreateFileA(portname,
                                                GENERIC_READ | GENERIC_WRITE,
                                                0,
                                                0,
                                                OPEN_EXISTING,
                                                FILE_ATTRIBUTE_NORMAL,
                                                0);
        if (hSerialPort[rbt->serialConnection] == INVALID_HANDLE_VALUE) return -1;



        dcbSerialParams.dcb.DCBlength=sizeof(dcbSerialParams);
        if (!GetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams.dcb)) return -2;

        dcbSerialParams.dcb.BaudRate = 115200;
        dcbSerialParams.dcb.ByteSize = 8;
        dcbSerialParams.dcb.StopBits = ONESTOPBIT;
        dcbSerialParams.dcb.Parity     = NOPARITY;

        comt.ReadIntervalTimeout = 500;
        SetCommTimeouts( hSerialPort[rbt->serialConnection], &comt);
        if (!SetCommState(hSerialPort[rbt->serialConnection], &dcbSerialParams.dcb)) return -3;

        rbt->port=comPort;
        rbt->connected=1;



        return 0;
}


int epuckSercomm::setRobotSpeed(struct robot *rbt, int speedL, int speedR)
{



        DWORD dwBytesReadWrite = 0;
        char command[5];

        command[0]='D';
    command[1]=speedL & 0xff;
    command[2]=speedL >> 8;
    command[3]=speedR & 0xff;
    command[4]=speedR >>8;

        if (!WriteFile(hSerialPort[rbt->serialConnection],command,5,&dwBytesReadWrite,NULL)) return -1;



        return 0;


}
int epuckSercomm::disconnectRobot(struct robot *rbt)
{

        CloseHandle(hSerialPort[rbt->serialConnection]);
        rbt->connected=0;


return 0;
}
int epuckSercomm::printRobotSensors(struct robot *rbt)
{
        printf("Robot no:%d\r\n",rbt->serialConnection);

        printf("IR: %d %d %d %d %d %d %d %d\r\n",
                rbt->IR[0],rbt->IR[1],rbt->IR[2],rbt->IR[3],
                rbt->IR[4],rbt->IR[5],rbt->IR[6],rbt->IR[7]);

        printf("LI: %d %d %d %d %d %d %d %d\r\n",
                rbt->LI[0],rbt->LI[1],rbt->LI[2],rbt->LI[3],
                rbt->LI[4],rbt->LI[5],rbt->LI[6],rbt->LI[7]);

        printf("FL: %d %d %d\r\n",
                rbt->FL[0],rbt->FL[1],rbt->FL[2]);

        printf("MC: %d %d %d\r\n",
                rbt->MC[0],rbt->MC[1],rbt->MC[2]);

return 0;

}
int  epuckSercomm::setRobotLed(struct robot *rbt,int noLed, int value)
{


        DWORD dwBytesReadWrite = 0;
        char command[3];
        if (value<0)value=0; //on
        if (value>1)value=1; //off

        command[0]='L';
    command[1]= noLed;   //number of led to switch
    command[2]= value;  //0: off, 1: on

        if (!WriteFile(hSerialPort[rbt->serialConnection],command,3,&dwBytesReadWrite,NULL))
                return -1;


        return 0;


}

To copy to clipboard, switch view to plain text mode 

.h FILE

#ifndef EPUCKSERCOMM_H
#define EPUCKSERCOMM_H
 
#include <QObject>
#include <stdio.h>
#include "mode.h"
#include "windows.h"
#include "time.h"
#include "defs.h"
 
class epuckSercomm : public QObject
{
public:
    epuckSercomm();
    int setRobotLed(struct robot *rbt,int noLed, int value);
    int printRobotSensors(struct robot *rbt);
    int disconnectRobot(struct robot *rbt);
    int setRobotSpeed(struct robot *rbt, int speedL, int speedR);
    int connectRobot(struct robot *rbt, int comPort);
    int initRobots(struct robot *rbt);
 
    //struct robot robots[7];
    HANDLE hSerialPort[7];  //up to 7 connection
    COMMCONFIG dcbSerialParams;
    COMMTIMEOUTS comt;
    // fine
    //stat
    clock_t startTime;
    clock_t endTime;
 
    int dtime;
};
 
 
#endif // EPUCKSERCOMM_H

#ifndef EPUCKSERCOMM_H
#define EPUCKSERCOMM_H

#include <QObject>
#include <stdio.h>
#include "mode.h"
#include "windows.h"
#include "time.h"
#include "defs.h"

class epuckSercomm : public QObject
{
public:
    epuckSercomm();
    int setRobotLed(struct robot *rbt,int noLed, int value);
    int printRobotSensors(struct robot *rbt);
    int disconnectRobot(struct robot *rbt);
    int setRobotSpeed(struct robot *rbt, int speedL, int speedR);
    int connectRobot(struct robot *rbt, int comPort);
    int initRobots(struct robot *rbt);

    //struct robot robots[7];
    HANDLE hSerialPort[7];  //up to 7 connection
    COMMCONFIG dcbSerialParams;
    COMMTIMEOUTS comt;
    // fine
    //stat
    clock_t startTime;
    clock_t endTime;

    int dtime;
};


#endif // EPUCKSERCOMM_H

To copy to clipboard, switch view to plain text mode 

and the initialization...

in the main UI program is

epuckSercomm *epuck = new epuckSercomm();
    epuck->initRobots(&robots[0]);
    epuck->connectRobot(&robots[1], 1036);
    epuck->setRobotLed(&robots[1],7,1);
    epuck->setRobotLed(&robots[1],7,0);

epuckSercomm *epuck = new epuckSercomm();
    epuck->initRobots(&robots[0]);
    epuck->connectRobot(&robots[1], 1036);
    epuck->setRobotLed(&robots[1],7,1);
    epuck->setRobotLed(&robots[1],7,0);

To copy to clipboard, switch view to plain text mode 

i am getting the following as my output error...i think it is something really that i am missing out... could you please point it out...
image.jpg

[wysota]

Ouch... first of all please preview your posts before you send them. Second of all don't inline large amount of code directly in your posts. Use attachments instead.

Your error is probably caused by the same file being referenced twice in the project file (or wrong files being there in the first place).

And really... your code is hardly readable because of all those pointers and dereferencing, why don't you clean it up a bit?

class RobotController {
public:
  RobotController(int rbtCnt = 7) {
    for(int i=0; i<rbtCnt; i++) {
      robot r;
      r.serialConnection = i;
      r.port = -1;
      r.connected = false;
      m_robots.append(r);
    }
  }
 
  bool connectRobot(int index, int port) {
    if(index<0 || index>=m_robots.count()) return;
    QString portName = QString("//./COM%1").arg(port);
    // seems that you can use QFile and QFileSystemWatcher or QSocketNotifier here but ok...
    m_handles[index] = CreateFileA(portName.toAscii().constData(), ...); 
    if(!m_handles[index]) {
      m_handles.remove(index);
      return false;
    }
    //...
    m_robots[index].port = port;
    m_robots[index].connected = true; // this is not needed, robot is connected if there is a handle for it or port != -1
    return true;
  }
 
  void disconnectRobot(int index) {
    if(!m_robots.contains(index)) return; // not connected
    CloseFile(m_handles.value(index));
    m_handles.remove(index);
    m_robots[index].port = -1;
    m_robots[index].connected = false; // again, not needed
  }
private:
  QList<robot> m_robots; // clean up "robot" class too
  QMap<int, HANDLE> m_handles;
};

class RobotController {
public:
  RobotController(int rbtCnt = 7) {
    for(int i=0; i<rbtCnt; i++) {
      robot r;
      r.serialConnection = i;
      r.port = -1;
      r.connected = false;
      m_robots.append(r);
    }
  }

  bool connectRobot(int index, int port) {
    if(index<0 || index>=m_robots.count()) return;
    QString portName = QString("//./COM%1").arg(port);
    // seems that you can use QFile and QFileSystemWatcher or QSocketNotifier here but ok...
    m_handles[index] = CreateFileA(portName.toAscii().constData(), ...); 
    if(!m_handles[index]) {
      m_handles.remove(index);
      return false;
    }
    //...
    m_robots[index].port = port;
    m_robots[index].connected = true; // this is not needed, robot is connected if there is a handle for it or port != -1
    return true;
  }

  void disconnectRobot(int index) {
    if(!m_robots.contains(index)) return; // not connected
    CloseFile(m_handles.value(index));
    m_handles.remove(index);
    m_robots[index].port = -1;
    m_robots[index].connected = false; // again, not needed
  }
private:
  QList<robot> m_robots; // clean up "robot" class too
  QMap<int, HANDLE> m_handles;
};

To copy to clipboard, switch view to plain text mode 

and so on...

[medampudi]

IT is true that many of the Classes in the robots and the other things needs cleaning up but they are the ones needed in my project for robot control... any how... the point is even after making all the changes you have suggested i am still getting the same error at the line 17 of the code shown above.. the HANDLE gives out a error always and leads to improper shutting down of the whole program.. While debugging it I have come across that the HANDLE returns a INVALID_HANDLE_VALUE always leading the program to shutdown improperly... is there any way i could use something for the CreateFileA instead and get the HANDLE working....

Rajesh...