ExternalVendorCode/trick
1
0
Fork 0
mirror of https://github.com/nasa/trick.git synced 2025-02-03 17:50:43 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

trick comm byteswap functions can't handle sizes above 65536 bytes

Browse Source 
Backporting fix that adds pthread specific swap buffer for both
tc_read and tc_write.

refs #187
This commit is contained in:
Alex Lin 2016-04-19 14:36:56 -05:00
parent be5835f92f
commit 0bd8c2f09e
3 changed files with 63 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
trick_source/trick_utils/comm/include/trick_byteswap.h
View File

@ -25,6 +25,11 @@ short trick_byteswap_short(short input);
}
#endif
struct SwapBuffer {
unsigned int size ;
char * swap_space ;
} ;
#define TRICK_GET_BYTE_ORDER(IND) \
{ \
union { \

34
trick_source/trick_utils/comm/src/tc_read_byteswap.c
View File

@ -4,14 +4,23 @@
* byte orders are different.
*/
#include <pthread.h>
#include "../include/tc.h"
#include "../include/tc_proto.h"
#include "../include/trick_byteswap.h"
static pthread_key_t key ;
static pthread_once_t key_once = PTHREAD_ONCE_INIT ;
static void make_key() {
pthread_key_create(&key, NULL) ;
}
int tc_read_byteswap(TCDevice * device, char *buffer, int size, ATTRIBUTES * attr)
{
char local_byteorder;
int ret = 0;
static char swap[65536];
struct SwapBuffer * swap_buffer ;
if (!device) {
TrickErrorHndlr *temp_error_hndlr = NULL;
@ -20,12 +29,27 @@ int tc_read_byteswap(TCDevice * device, char *buffer, int size, ATTRIBUTES * att
return (-1);
}
pthread_once(&key_once, make_key) ;
if (( swap_buffer = pthread_getspecific(key)) == NULL ) {
swap_buffer = malloc(sizeof(struct SwapBuffer)) ;
swap_buffer->size = 0 ;
pthread_setspecific(key, swap_buffer) ;
}
if ( (unsigned int)size > swap_buffer->size ) {
unsigned int new_size = (unsigned int)size ;
if ( new_size % 4 ) {
new_size = new_size - ( new_size % 4 ) + 4 ;
}
swap_buffer->swap_space = malloc(new_size) ;
swap_buffer->size = new_size ;
}
TRICK_GET_BYTE_ORDER(local_byteorder)
if (device->byte_info[TC_BYTE_ORDER_NDX] != local_byteorder) {
memset(swap, 0, (size_t) size);
ret = tc_read(device, (char *) swap, size);
trick_bswap_buffer(buffer, swap, attr, 0);
if (device->byte_info[TC_BYTE_ORDER_NDX] != local_byteorder) {
memset(swap_buffer->swap_space, 0, (size_t) size);
ret = tc_read(device, (char *) swap_buffer->swap_space, size);
trick_bswap_buffer(buffer, swap_buffer->swap_space, attr, 0);
return (ret);
} else {
return (tc_read(device, (char *) buffer, size));

34
trick_source/trick_utils/comm/src/tc_write_byteswap.c
View File

@ -4,14 +4,23 @@
* byte orders are different.
*/
#include <pthread.h>
#include "../include/tc.h"
#include "../include/tc_proto.h"
#include "../include/trick_byteswap.h"
static pthread_key_t key ;
static pthread_once_t key_once = PTHREAD_ONCE_INIT ;
static void make_key() {
pthread_key_create(&key, NULL) ;
}
int tc_write_byteswap(TCDevice * device, char *buffer, int size, ATTRIBUTES * attr)
{
char local_byteorder;
int ret = 0;
static char swap[65536];
struct SwapBuffer * swap_buffer ;
if (!device) {
TrickErrorHndlr *temp_error_hndlr = NULL;
@ -20,12 +29,27 @@ int tc_write_byteswap(TCDevice * device, char *buffer, int size, ATTRIBUTES * at
return (-1);
}
pthread_once(&key_once, make_key) ;
if (( swap_buffer = pthread_getspecific(key)) == NULL ) {
swap_buffer = malloc(sizeof(struct SwapBuffer)) ;
swap_buffer->size = 0 ;
pthread_setspecific(key, swap_buffer) ;
}
if ( (unsigned int)size > swap_buffer->size ) {
unsigned int new_size = (unsigned int)size ;
if ( new_size % 4 ) {
new_size = new_size - ( new_size % 4 ) + 4 ;
}
swap_buffer->swap_space = malloc(new_size) ;
swap_buffer->size = new_size ;
}
TRICK_GET_BYTE_ORDER(local_byteorder)
if (device->byte_info[TC_BYTE_ORDER_NDX] != local_byteorder) {
memset(swap, 0, (size_t) size);
trick_bswap_buffer(swap, buffer, attr, 1);
ret = tc_write(device, (char *) swap, size);
if (device->byte_info[TC_BYTE_ORDER_NDX] != local_byteorder) {
memset(swap_buffer->swap_space, 0, (size_t) size);
trick_bswap_buffer(swap_buffer->swap_space, buffer, attr, 1);
ret = tc_write(device, (char *) swap_buffer->swap_space, size);
return (ret);
} else {
return (tc_write(device, (char *) buffer, size));