<html><head><meta http-equiv="content-type" content="text/html; charset=UTF-8"><style>body { line-height: 1.5; }blockquote { margin-top: 0px; margin-bottom: 0px; margin-left: 0.5em; }body { font-size: 14px; font-family: 'Microsoft YaHei UI'; color: rgb(0, 0, 0); line-height: 1.5; }</style></head><body>
<div><span></span>thanks for good question.</div><div><br></div><div>i check the source linux-source-5.8.0,in filesecurity/smack/smack_lsm.c:<span style="line-height: 1.5; background-color: transparent;">smack_getprocattr</span></div><div>directly return <b style="line-height: 1.5; background-color: transparent;">-EINVAL.</b></div><div><br></div><div><div>static int smack_getprocattr(struct task_struct *p, char *name, char **value)</div><div>{</div><div> struct smack_known *skp = smk_of_task_struct(p);</div><div> char *cp;</div><div> int slen;</div><div><br></div><div><b> if (strcmp(name, "current") != 0 && strcmp(name, "context") != 0)</b></div><div><b> return -EINVAL;</b></div></div><div><br></div><div><div> cp = kstrdup(skp->smk_known, GFP_KERNEL);</div><div> if (cp == NULL)</div><div> return -ENOMEM;</div><div><br></div><div> slen = strlen(cp);</div><div> *value = cp;</div><div> return slen;</div><div>}</div></div><div><br></div>
<div><br></div>
<blockquote style="margin-Top: 0px; margin-Bottom: 0px; margin-Left: 0.5em; margin-Right: inherit"><div> </div><div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm"><div style="PADDING-RIGHT: 8px; PADDING-LEFT: 8px; FONT-SIZE: 12px;FONT-FAMILY:tahoma;COLOR:#000000; BACKGROUND: #efefef; PADDING-BOTTOM: 8px; PADDING-TOP: 8px"><div><b>From:</b> <a href="mailto:jstancek@redhat.com">Jan Stancek</a></div><div><b>Date:</b> 2021-01-15 22:54</div><div><b>To:</b> <a href="mailto:liuxp11@chinatelecom.cn">Xinpeng Liu</a></div><div><b>CC:</b> <a href="mailto:ltp@lists.linux.it">ltp</a></div><div><b>Subject:</b> Re: [LTP] [PATCH] fs/proc01.c:add known issues</div></div></div><div><div> </div>
<div>----- Original Message -----</div>
<div>> Test in ubuntu20.10,there are several failure tests.</div>
<div>> </div>
<div>> proc01 1 TFAIL : proc01.c:396: read failed:</div>
<div>> /proc/self/task/61595/attr/smack/current: errno=EINVAL(22): Invalid argument</div>
<div>> proc01 2 TFAIL : proc01.c:396: read failed:</div>
<div>> /proc/self/task/61595/attr/apparmor/prev: errno=EINVAL(22): Invalid argument</div>
<div>> proc01 3 TFAIL : proc01.c:396: read failed:</div>
<div>> /proc/self/task/61595/attr/apparmor/exec: errno=EINVAL(22): Invalid argument</div>
<div>> proc01 4 TFAIL : proc01.c:396: read failed:</div>
<div>> /proc/self/attr/smack/current: errno=EINVAL(22): Invalid argument</div>
<div>> proc01 5 TFAIL : proc01.c:396: read failed:</div>
<div>> /proc/self/attr/apparmor/prev: errno=EINVAL(22): Invalid argument</div>
<div>> proc01 6 TFAIL : proc01.c:396: read failed:</div>
<div>> /proc/self/attr/apparmor/exec: errno=EINVAL(22): Invalid argument</div>
<div> </div>
<div>I'm OK with ignoring it, but commit log could explain more why we get EINVAL</div>
<div>on read here.</div>
<div> </div>
</div></blockquote>
</body></html>