Re: [PATCH v2 2/3] fs/binfmts: Better handling of binfmt loops

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 08/02/2013 03:12 PM, Zach Levis wrote:

+		if (retval == -ELOOP && bprm->recursion_depth == 0) { /* cur, previous */
+			pr_err("Too much recursion with binfmts (0:%s, -1:%s) in file %s, skipping (base %s).\n",
+					bprm->previous_binfmts[0]->name,
+					bprm->previous_binfmts[1]->name,
+					bprm->filename,
+					fmt->name);
+
+			/* Put argv back in its place */
+			bprm->p = bprm->p_no_argv;
+
+			bprm->argc = count(*(bprm->argv_orig), MAX_ARG_STRINGS);
+			retval = copy_strings(bprm->argc, *(bprm->argv_orig), bprm);
+			if (retval < 0)
+				return retval;
+
+			retval = -ENOEXEC;
+			continue;
+		}
  	}
  	read_unlock(&binfmt_lock);


NEVERMIND, I messed up the order here so this doesn't work! v3 coming up without the stupid.

--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux