This is an archive of the discontinued LLVM Phabricator instance.

Added check for malloc return
ClosedPublic

Authored by vhscampos on Nov 4 2016, 5:04 PM.

Reviewers

Commits

Summary

This malloc() didn't have a check for successfully allocated memory. I created one following other examples.

vhscampos retitled this revision from to Added check for malloc return.

vhscampos updated this object.

LGTM

runtime/src/kmp_i18n.c
839 ↗	(On Diff #76956)	Please add same check here as well.

This revision is now accepted and ready to land.Nov 8 2016, 10:44 AM

Done.

Victor,
Shall I commit the changes on your behalf?

Yes, please.

This revision was automatically updated to reflect the committed changes.

Path

Size

openmp/

trunk/

runtime/

src/

6 lines

Show First 20 Lines • Show All 821 Lines • ▼ Show 20 Lines	#else // Non-Windows* OS: Linux* OS or OS X*
#else // OS X, FreeBSD etc.		#else // OS X, FreeBSD etc.

// XSI version of strerror_r.		// XSI version of strerror_r.

int size = 2048;		int size = 2048;
// TODO: Add checking result of malloc().		// TODO: Add checking result of malloc().
char * buffer = (char *) KMP_INTERNAL_MALLOC( size );		char * buffer = (char *) KMP_INTERNAL_MALLOC( size );
int rc;		int rc;
		if (buffer == NULL) {
		KMP_FATAL(MemoryAllocFailed);
		}
rc = strerror_r( err, buffer, size );		rc = strerror_r( err, buffer, size );
if ( rc == -1 ) {		if ( rc == -1 ) {
rc = errno; // XSI version sets errno.		rc = errno; // XSI version sets errno.
}; // if		}; // if
while ( rc == ERANGE ) { // ERANGE means the buffer is too small.		while ( rc == ERANGE ) { // ERANGE means the buffer is too small.
KMP_INTERNAL_FREE( buffer );		KMP_INTERNAL_FREE( buffer );
size *= 2;		size *= 2;
buffer = (char *) KMP_INTERNAL_MALLOC( size );		buffer = (char *) KMP_INTERNAL_MALLOC( size );
		if (buffer == NULL) {
		KMP_FATAL(MemoryAllocFailed);
		}
rc = strerror_r( err, buffer, size );		rc = strerror_r( err, buffer, size );
if ( rc == -1 ) {		if ( rc == -1 ) {
rc = errno; // XSI version sets errno.		rc = errno; // XSI version sets errno.
}; // if		}; // if
}; // while		}; // while
if ( rc == 0 ) {		if ( rc == 0 ) {
message = buffer;		message = buffer;
} else {		} else {
▲ Show 20 Lines • Show All 135 Lines • Show Last 20 Lines