Изпращане на AT команди към USR модем в C

Чудех се дали някой от вас знае какво правя грешно тук? Така че имам тази програма на C, която изпраща AT команди към модем. Тези команди са тествани на хипертерминал и работят добре, но когато ги изпратя през модема, получавам първо и „OK“ за първата команда „AT“, което е добре, но след това, при изпращане на следващата команда, модемът отговаря с "AT+CC"...което нямам идея какво означава. Всяка помощ се оценява.

източник:

void sendSMS(const char* port, const char* number, const char* baud)
{
    HANDLE hComm;
    DCB dcb;
    BOOL fSuccess;

    hComm = CreateFile(port,                //port name
        GENERIC_READ | GENERIC_WRITE, //Read/Write
        0,                            // No Sharing
        NULL,                         // No Security
        OPEN_EXISTING,// Open existing port only
        0,            // Non Overlapped I/O
        NULL);        // Null for Comm Devices

    if (hComm == INVALID_HANDLE_VALUE)
    {
        printf("ERROR: Cannot open serial port\r\n");
        return;
    }
    else
        printf("STATUS: Serial port opened\r\n");


    // Configure PORT
    //  Initialize the DCB structure.
    SecureZeroMemory(&dcb, sizeof(DCB));
    dcb.DCBlength = sizeof(DCB);

    //  Build on the current configuration by first retrieving all current
    //  settings.
    fSuccess = GetCommState(hComm, &dcb);

    if (!fSuccess)
    {
        //  Handle the error.
        printf("GetCommState failed with error %d.\n", GetLastError());
        return;
    }
                               //  Fill in some DCB values and set the com state: 
                               //  57,600 bps, 8 data bits, no parity, and 1 stop bit.
    dcb.BaudRate = atoi(baud);     //  baud rate
    dcb.ByteSize = 8;             //  data size, xmit and rcv
    dcb.Parity = NOPARITY;      //  parity bit
    dcb.StopBits = ONESTOPBIT;    //  stop bit
    dcb.fOutxCtsFlow = FALSE;
    dcb.fOutxDsrFlow = FALSE;
    dcb.fOutX = FALSE;

    fSuccess = SetCommState(hComm, &dcb);

    if (!fSuccess)
    {
        //  Handle the error.
        printf("SetCommState failed with error %d.\n", GetLastError());
        return;
    }

    //  Get the comm config again.
    fSuccess = GetCommState(hComm, &dcb);

    if (!fSuccess)
    {
        //  Handle the error.
        printf("GetCommState failed with error %d.\n", GetLastError());
        return;
    }

    // End Config

    sendATCommands(hComm, number, "This is NOT a test.");
    CloseHandle(hComm);//Closing the Serial Port

}

void sendATCommands(HANDLE hComm, const char* number, const char message[])
{
    char str[256];

    if (!writeToPort(hComm, "AT\r\n")) //Hello modem
        return;

    if (!readFromPort(hComm)) // Must be OK
        return;

    if (!writeToPort(hComm, "AT+CMGF=1\r\n")) //Modem, prepare to send text messages
        return;

    if (!readFromPort(hComm)) // Must be OK again
        return;

    memset(str, 0, 256); 
    strcpy_s(str, "AT+CMGS=\"");
    strcat_s(str, 256, number);
    strcat_s(str, 256, "\"\r\n");

    if (!writeToPort(hComm, str)) //Modem, here's the number to send the message to
        return;

    if (!readFromPort(hComm)) // Must be ">" 
        return;

    memset(str, 0, 256); 
    strcpy_s(str, message);
    strcat_s(str, 256, "^Z");

    if (!writeToPort(hComm, str)) //Modem, communicate this to the number I gave you.
        return;

    if (!readFromPort(hComm)) // Must be CMGS: ##
        return;
}


int writeToPort(HANDLE hComm, const char lpBuffer[])
{
    DWORD dNoOFBytestoWrite;         // No of bytes to write into the port
    DWORD dNoOfBytesWritten = 0;     // No of bytes written to the port
    dNoOFBytestoWrite = sizeof(lpBuffer);

    int Status = WriteFile(hComm,        // Handle to the Serial port
                        lpBuffer,     // Data to be written to the port
                dNoOFBytestoWrite,  //No of bytes to write
                &dNoOfBytesWritten, //Bytes written
                            NULL);

    if (Status == FALSE)
    {
        printf("ERROR: Cannot write to serial port\r\n");
    }
    else
        printf("STATUS: Command %s \n written to port.\r\n", lpBuffer); 

    return Status;
}

int readFromPort(HANDLE hComm)
{
    char TempChar; //Temporary character used for reading
    char SerialBuffer[256];//Buffer for storing Rxed Data
    DWORD NoBytesRead = 0;
    int i = 0;
    int status;

    memset(SerialBuffer, 0, 256);
    printf("STATUS: Waiting response...\r\n");

    do
    {
        status = ReadFile(hComm,           //Handle of the Serial port
            &TempChar,       //Temporary character
            sizeof(TempChar),//Size of TempChar
            &NoBytesRead,    //Number of bytes read
            NULL);

        if (!status)
        {
            printf("ERROR: Cannot read from serial port\r\n");
            break;
        }

        SerialBuffer[i] = TempChar;// Store Tempchar into buffer
        i++;
    }
    while (NoBytesRead > 0);

    if (status)
        printf("PORT RESPONSE: %s \r\n", SerialBuffer);

    return status;
}

person Sergio Flores    schedule 01.03.2018    source източник
comment
Прегледайте strcpy_s(str, "AT+CMGS=\""); strcpy_s(str, message); и колко безопасен е този код. След това активирайте всички предупреждения на компилатора.   -  person chux - Reinstate Monica    schedule 01.03.2018
comment
благодаря, но получавам грешката от втората изпратена AT команда, която е твърдо кодирана, тази, която гласи AT+CMFG=1\r\n. Така че първата команда, която е обикновена AT\r\n, работи добре и модемът отговаря OK, но при втората команда, тази, която току-що казах, получавам обратно AT+CC ... ... аз полудявам тук   -  person Sergio Flores    schedule 01.03.2018
comment
Не съм сигурен, че \r\n е правилен. Вярвам, че натискането на enter в хипертерминал през този интерфейс просто изпраща AT. Мина известно време, но опитайте да премахнете \r\n? ^M или някакъв друг знак може да е необходим, но \r\n не изглежда правилно. Може и да греша   -  person zzxyz    schedule 01.03.2018
comment
Какъв C компилатор използвате?   -  person chux - Reinstate Monica    schedule 01.03.2018
comment
Благодаря, да, но ако е грешно, тогава защо получавам първото ОК обратно? Освен това опитах AT (без CR), AT\r и те винаги водят до връщане на странен знак, ASCII 204... което според мен означава ИЗКЛЮЧВАНЕ. Може ли да е проблем с функцията readPort?   -  person Sergio Flores    schedule 01.03.2018
comment
Използвам VS community 2017, при многобайтова настройка   -  person Sergio Flores    schedule 01.03.2018
comment
strcat_s(str, 256, "^Z"); е подозрителен. Наистина ли искате да добавите 2 знака тук или 1?   -  person chux - Reinstate Monica    schedule 01.03.2018
comment
Добра точка. Трябва да изпратя там ‹CTRL-Z›. Отново обаче, грешката възниква доста преди да стигнем до този ред...   -  person Sergio Flores    schedule 01.03.2018
comment
Краят на отговор на AT команда може да е полезен.   -  person chux - Reinstate Monica    schedule 01.03.2018
comment
Кои са някои примерни низове, предадени на writeToPort(hComm, str) веднага след strcpy_s(str, "AT+CMGS=\""); strcat_s(str, 256, number); strcat_s(str, 256, "\"\r\n");? Уверете се, че не конструирате мислено низа, а извличате чрез отстраняване на грешки.   -  person chux - Reinstate Monica    schedule 01.03.2018
comment
От паметта (нямам лаптопа си под ръка, за да го тествам), но второто повикване ще повтори командата с текущата й настройка, ако устройството ви е конфигурирано да връща кодове с разширени резултати. Проверете ръководството си за устройството, потърсете командата AT, за да изключите разширените кодове за резултати, след което трябва да откриете, че просто получавате ОК обратно, както се очаква.   -  person shawty    schedule 01.03.2018
comment
Благодаря, добре, всъщност не ме интересува отговорът на модема, стига да не е грешка. Проверих низовете и изглежда добре, например: AT+CMGS=\99999999\\r\n -› Очаква се.   -  person Sergio Flores    schedule 02.03.2018
comment
И това НЕ е тест.^Z -› се очаква... Но това ли е правилният начин за изпращане на CTRL-Z? Благодаря   -  person Sergio Flores    schedule 02.03.2018