rebase

[Abhinavpv28]

No description provided.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
6 out of 8 committers have signed the CLA.

✅ Saranya2421
✅ nhanasi
✅ AravindanNC
✅ Abhinavpv28
✅ venkat0557
✅ Alan-Ryan
❌ Saranya
❌ naveenkumarhanasi

---

Saranya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[Copilot]

Pull request overview

Copilot reviewed 63 out of 63 changed files in this pull request and generated 11 comments.

[Copilot]

This test module performs file I/O and mutates /etc/rrd/remote_debugger.json at import time. Pytest will execute this on collection, which can make test runs order-dependent and can affect other tests (and local environments) even if these tests are skipped/xfail. Move this JSON read/update/write into a dedicated test/fixture (e.g., setup function) and ensure the original file contents are restored in teardown/finalizer.

[Copilot]

SCRIPT_SUCCESS / SCRIPT_FAILURE constants are swapped: SCRIPT_SUCCESS is set to the "...upload Failed" message and SCRIPT_FAILURE to the "...upload Success" message, and the printed status strings are inverted as well. This will report success on failure and vice versa. Swap the constants (or adjust the condition/print statements) so the labels match the expected log lines.

[Copilot]

The CalledProcessError from running create_json.sh is caught and only printed, so the test will continue and potentially pass even when the script fails. Re-raise the exception or fail the test (e.g., assert False) in the except block so environment setup failures are not silently ignored.

[Copilot]

rrd_upload_cleanup_source_dir builds an "rm -rf" shell command with an unquoted dir_path and executes it via system(). Since dir_path is derived from issue/category values that ultimately come from external input, this is vulnerable to command injection and also breaks paths containing spaces/shell metacharacters. Prefer a safe recursive delete using nftw()/unlinkat() (or, at minimum, execv("rm", {"rm","-rf","--",dir_path,...}) without invoking a shell).

[Copilot]

In the GTEST_ENABLE build, this block skips freeing cmdData->rfcvalue/cmdData->command/cmdData entirely, which introduces a leak in unit tests and makes behavior differ between test and production builds. Consider guarding only the systemctl calls with GTEST_ENABLE, but keep the memory cleanup outside the conditional.

[Copilot]

The ustar name/prefix split logic for paths 101–255 chars is incorrect: it splits at an arbitrary offset (name_len - 101) rather than at the last '/' boundary. This can produce invalid tar headers and lose path information. Split the path at the last '/' such that prefix <= 155 and name <= 100 per ustar rules, or reject names that can’t be represented correctly.

Suggested change

	if (name_len <= 100) {
	strncpy(hdr.name, name, sizeof(hdr.name) - 1);
	hdr.name[sizeof(hdr.name) - 1] = '\0';
	} else if (name_len <= 255) {
	/* split into prefix and name */
	size_t prefix_len = name_len - 100 - 1; /* leave room for null */
	if (prefix_len > sizeof(hdr.prefix)) {
	RDK_LOG(RDK_LOG_ERROR, LOG_REMDEBUG, "[%s] File name too long: %s (length %zu)\n",
	__FUNCTION__, name, name_len);
	return -1;
	}
	strncpy(hdr.prefix, name, prefix_len);
	hdr.prefix[prefix_len < sizeof(hdr.prefix) ? prefix_len : sizeof(hdr.prefix) - 1] = '\0';
	strncpy(hdr.name, name + prefix_len + 1, sizeof(hdr.name) - 1);
	hdr.name[sizeof(hdr.name) - 1] = '\0';
	if (name_len <= sizeof(hdr.name)) {
	/* simple case: whole path fits in name field */
	strncpy(hdr.name, name, sizeof(hdr.name) - 1);
	hdr.name[sizeof(hdr.name) - 1] = '\0';
	} else if (name_len <= 255) {
	/* split into prefix and name according to POSIX ustar rules:
	* - prefix contains the directory path (no trailing '/')
	* - name contains the final path component
	* - both fields must fit in their respective header buffers
	*/
	size_t prefix_len = 0;
	size_t name_part_len = 0;
	int split_found = 0;
	/* search for last '/' that yields valid prefix and name lengths */
	for (size_t i = name_len; i > 0; --i) {
	if (name[i - 1] == '/') {
	prefix_len = i - 1; /* up to but not including this '/' */
	name_part_len = name_len - i; /* after '/' to end */
	if (prefix_len > 0 &&
	prefix_len <= sizeof(hdr.prefix) &&
	name_part_len > 0 &&
	name_part_len <= sizeof(hdr.name)) {
	split_found = 1;
	break;
	}
	}
	}
	if (!split_found) {
	RDK_LOG(RDK_LOG_ERROR, LOG_REMDEBUG,
	"[%s] File name too long or invalid for ustar: %s (length %zu)\n",
	__FUNCTION__, name, name_len);
	return -1;
	}
	/* copy prefix (directory path) */
	memcpy(hdr.prefix, name, prefix_len);
	hdr.prefix[prefix_len] = '\0';
	/* copy final path component */
	memcpy(hdr.name, name + prefix_len + 1, name_part_len);
	hdr.name[name_part_len] = '\0';

[Copilot]

rrd_upload.h declares rrd_upload_invoke_logupload_api() and rrd_upload_cleanup(), but there are no implementations in the source tree. Since this header is installed as part of the public include set, this creates an API contract that will fail at link time for consumers. Either implement these functions or remove them from the public header until they exist.

[Copilot]

rrd_log.h declares rrd_log_init(), but there is no implementation in the repository. If this is intended as a public API, it should be implemented; otherwise, remove the declaration to avoid unresolved symbol/link-time failures for any consumer that includes this installed header.

[Copilot]

Typo in step text: "shopuld" should be "should".

[Copilot]

Typo in step text: "shopuld" should be "should".

[Copilot]

Pull request overview

Copilot reviewed 81 out of 81 changed files in this pull request and generated 5 comments.

[Copilot]

Pull request overview

Copilot reviewed 83 out of 84 changed files in this pull request and generated 6 comments.

[Copilot]

Pull request overview

Copilot reviewed 88 out of 89 changed files in this pull request and generated 8 comments.

[Copilot]

Pull request overview

Copilot reviewed 89 out of 90 changed files in this pull request and generated 3 comments.