Ошибка памяти в LINUX, но не в UNIX - PullRequest
2 голосов
/ 12 октября 2011

Мы запускаем скрипт ksh, который вызывает программу ProC, которая обновляет таблицу. Программа успешно работает в UNIX, но когда мы запускаем ее в LINUX RHEL55, она вызывает ошибку памяти Когда мы пытались отладить, где последний оператор до ошибки сбоя памяти, он указывает на точку после того, как мы делаем выборку на курсоре и пытается использовать значения, выбранные для обновления таблицы Вот фрагмент кода

#define PROGRAM  "n377wlocgeo"    /* name of this program */

#define USERNAME "/"               /* ORACLE username */

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <time.h>


   /* Include the ORACLE communication Area, a structure through which
   ORACLE makes additional runtime Status available to the program
   The ORACA=YES must be specified to enable use of oraca */

   EXEC SQL INCLUDE ORACA;
   EXEC ORACLE OPTION (ORACA=YES);

   EXEC SQL INCLUDE SQLCA;


/*---------------------------------------------------------------------------
 WORKING STORAGE */

/* GLOBAL EXTERNAL VARIABLES */


char *ptr;                 /* pointer for record layout */
char timestamp[26] = "\0"; /* current date and time */


/* Counters */
int   rec_err_ctr = 0;
int   cnt_rec = 0;

long  update_no_ctr = 0;
long  update_geo_ctr = 0;

long  update_no_pr_ctr = 0;
long  update_no_us_ctr = 0;

int   no_fetch_rec_ctr = 0;
int   geo_fetch_rec_ctr = 0;

int   commit_ctr = 0;


int  ws_err_sqlcode;
char para_name[25];         /* hold for displaying what paragraph abended */
char err_para_name[25];     /* hold for displaying what paragraph abended */
char abend_msg[240];


/*Pgm Control Vars */

char  error_in_job = 'N';
char  no_fetch_ended = 'N';
char  geo_fetch_ended = 'N';
char  clear_psl_cd[12];



 /* UNIX ENVIRONMENT VARIABLES  */
char debug[2];

 /* Function delarations */
 void initpara();        /* connect to Oracle  */
 void get_env_vars();    /* get the unix environment variables into C program */
 void get_timestamp();   /* get the date and time from the system */

 void connect_oracle();
 void disconnect_oracle();

 void get_update_timestamp();

 void end_transaction();
 void error_routine();
 void echofields();           /* for debugging, echo all fields parsed */

 void no_geo_cursor_process();
 void geo_cursor_process();
 void sql_no_update();
 void sql_geo_update();
 void sql_no_pr_update();
 void sql_no_us_update();



EXEC SQL BEGIN DECLARE SECTION;


 varchar      hv_psl_cd[12];
 varchar      hv_rec_udt_ts[20];
 varchar      hv_geo_cny_cd[03];
 varchar      hv_geo_psl_cd[12];
 varchar      hv_geo_typ_cd[4];
 varchar      hv_cny_cd[03];


/* Cursor to set defaults for countries with no geo classification */
 EXEC SQL DECLARE NO_GEO_CUR CURSOR FOR
      SELECT   CNY_CD
              ,PSL_CD
      FROM TDLOCTN_BASE
      WHERE CNY_CD NOT IN ('US', 'PR')
          AND GEO_ARA_PSL_CSF_CD is null
      GROUP BY CNY_CD, PSL_CD
      ORDER BY CNY_CD, PSL_CD;


  EXEC SQL DECLARE GEO_CUR CURSOR FOR
      SELECT GEO_CNY_CD,
             GEO_PSL_CD,
             GEO_TYP_CD
      FROM TDLOC_GEO_CD_EXT
      GROUP BY GEO_CNY_CD,GEO_PSL_CD,GEO_TYP_CD
      ORDER BY GEO_CNY_CD,GEO_PSL_CD;


EXEC SQL END DECLARE SECTION;

/*----------------------------------------------------------
 PROCEDURE DIVISION
------------------------------------------------------------
   MAINLINE */

/*------------------------------------------------------------*/
 int main(int argc, char **argv)

   {


      printf ("Starting");

      get_timestamp();

      printf("PGM BEGIN DATE/TIME : %s \n", timestamp );

     initpara();

     if ( strcmp(debug, "Y") == 0 )
           get_timestamp();
           printf("main while loop: %s\n", timestamp);

      /* get max rec_udt_tms for cursor process */
      get_update_timestamp();

      /* open the cursor and fetch all rows for defaults  . */

      no_geo_cursor_process();

     if ( error_in_job == 'Y' )
          exit(2);
      else
          exit(0);
   }


/*------------------------------------------------------------*/
  void get_timestamp()

  {
    strcpy(para_name, "Para = get_timestamp");
    if ( strcmp(debug, "Y") == 0 )
           printf("function: get_timestamp\n");

    struct tm *tm_now;
    time_t secs_now;

/no_geo_cursor

    EXEC SQL
        SELECT TRUNC(MAX(REC_UDT_TS))
               INTO :hv_rec_udt_ts
         FROM TDLOCTN_BASE;



    if ( sqlca.sqlcode == 0 )
        printf("CHAR  %-25s : %s \n", "hv_rec_udt_ts", hv_rec_udt_ts.arr);
    else
       {
         printf("SQL Select Error in - get_update_timestamp: %s\n");
         strcpy(abend_msg, sqlca.sqlerrm.sqlerrmc);
         printf("SQL ERROR CODE: %d\n", sqlca.sqlcode);
         printf("SQL ERROR MSG: %s\n", abend_msg);
         error_routine();
        }
  }

 void no_geo_cursor_process ()

   {
     strcpy(para_name, "Para = no_geo_cursor_process");
     if ( strcmp(debug, "Y") == 0 )
          printf("function: no_geo_cursor_process\n");

     EXEC SQL
        OPEN NO_GEO_CUR;

     if ( sqlca.sqlcode == 0 )
        printf("Cursor Opened: %s\n");
     else
        {
         printf("SQL Open Error in - no_geo_cursor_process: %s\n");
         strcpy(abend_msg, sqlca.sqlerrm.sqlerrmc);
         printf("SQL ERROR CODE: %d\n", sqlca.sqlcode);
         printf("SQL ERROR MSG: %s\n", abend_msg);
         error_routine();
        }

1 Ответ

4 голосов
/ 12 октября 2011
char para_name[25];
strcpy(para_name, "Para = no_geo_cursor_process");
//                          1         2
//                 12345678901234567890123456789 (one extra for the NULL).

Это одна вещь, которая, вероятно, не будет работать.Это неопределенное поведение, если вы пишете после конца массива (вы не можете положить двадцать девять бисквитов в форму для печенья, построенную за двадцать пять (a) )

Увеличить размер1008 * для решения этой проблемы.

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


Кроме того, как указано в комментариях, у вас есть несколько строк, в которых вы предоставляете строки формата printf без соответствующих значений:

printf("SQL Select Error in - get_update_timestamp: %s\n");
printf("Cursor Opened: %s\n");

Это также неопределенное поведение и, вероятно, вызовет проблемы, обычно с использованием любых значений, которые были помещены в стек в качестве указателя строки.Вы можете просто удалить ошибочную %s из строки или, альтернативно, выяснить, какую строку вы хотите включить в сообщение, и добавить ее к аргументам printf.


(а) Если, конечно, вы не хотите, чтобы печенье рухнуло: -)

...