Merge "Make IEffect command more robust (second try)" into klp-dev (ab203130) · Commits · e / devices / smdk4412 / android_frameworks_av

media/libmedia/IEffect.cpp

+20 −7

Original line number	Diff line number	Diff line
		@@ -85,13 +85,15 @@ public:
		data.writeInt32(size);

		status_t status = remote()->transact(COMMAND, data, &reply);
		if (status == NO_ERROR) {
		status = reply.readInt32();
		}
		if (status != NO_ERROR) {
		if (pReplySize != NULL)
		*pReplySize = 0;
		return status;
		}

		status = reply.readInt32();
		size = reply.readInt32();
		if (size != 0 && pReplyData != NULL && pReplySize != NULL) {
		reply.read(pReplyData, size);
		@@ -152,6 +154,10 @@ status_t BnEffect::onTransact(
		char *cmd = NULL;
		if (cmdSize) {
		cmd = (char *)calloc(cmdSize, 1);
		if (cmd == NULL) {
		reply->writeInt32(NO_MEMORY);
		return NO_ERROR;
		}
		data.read(cmd, cmdSize);
		}
		uint32_t replySize = data.readInt32();
		@@ -159,9 +165,15 @@ status_t BnEffect::onTransact(
		char *resp = NULL;
		if (replySize) {
		resp = (char *)calloc(replySize, 1);
		if (resp == NULL) {
		free(cmd);
		reply->writeInt32(NO_MEMORY);
		return NO_ERROR;
		}
		}
		status_t status = command(cmdCode, cmdSize, cmd, &replySz, resp);
		reply->writeInt32(status);
		if (status == NO_ERROR) {
		if (replySz < replySize) {
		replySize = replySz;
		}
		@@ -169,6 +181,7 @@ status_t BnEffect::onTransact(
		if (replySize) {
		reply->write(resp, replySize);
		}
		}
		if (cmd) {
		free(cmd);
		}