Re: audit function doc. question

2005-07-06 Thread Chris Wright
* randy_dunlap ([EMAIL PROTECTED]) wrote:
> Yes, that's why I asked, I'm adding kerneldoc format comments
> to audit*.c (2 files).  You'll see it soon.

Great!  Thanks Randy.
-chris
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: audit function doc. question

2005-07-06 Thread Chris Wright
* randy_dunlap ([EMAIL PROTECTED]) wrote:
> kernel/audit.c (2.6.13-rc1-git5) audit_log_start() says:
> 
> /* Obtain an audit buffer.  This routine does locking to obtain the
>  * audit buffer, but then no locking is required for calls to
>  * audit_log_*format.  If the tsk is a task that is currently in a
>  * syscall, then the syscall is marked as auditable and an audit record
>  * will be written at syscall exit.  If there is no associated task, tsk
>  * should be NULL. */
> struct audit_buffer *audit_log_start(struct audit_context *ctx, int type)
> {
> 
> What does  refer to in the function description?
> There is no  in this function.

It refers to tsk, tsk, stale comment.  It's task->audit_context (which is
ctx there).  Interested in preparing a patch, could even move to proper
kerneldoc format ;-)

thanks,
-chris
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: audit function doc. question

2005-07-06 Thread randy_dunlap
On Wed, 6 Jul 2005 15:21:15 -0700 Chris Wright wrote:

| * randy_dunlap ([EMAIL PROTECTED]) wrote:
| > kernel/audit.c (2.6.13-rc1-git5) audit_log_start() says:
| > 
| > /* Obtain an audit buffer.  This routine does locking to obtain the
| >  * audit buffer, but then no locking is required for calls to
| >  * audit_log_*format.  If the tsk is a task that is currently in a
| >  * syscall, then the syscall is marked as auditable and an audit record
| >  * will be written at syscall exit.  If there is no associated task, tsk
| >  * should be NULL. */
| > struct audit_buffer *audit_log_start(struct audit_context *ctx, int type)
| > {
| > 
| > What does  refer to in the function description?
| > There is no  in this function.
| 
| It refers to tsk, tsk, stale comment.  It's task->audit_context (which is
| ctx there).  Interested in preparing a patch, could even move to proper
| kerneldoc format ;-)

Yes, that's why I asked, I'm adding kerneldoc format comments
to audit*.c (2 files).  You'll see it soon.

---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


audit function doc. question

2005-07-06 Thread randy_dunlap
kernel/audit.c (2.6.13-rc1-git5) audit_log_start() says:

/* Obtain an audit buffer.  This routine does locking to obtain the
 * audit buffer, but then no locking is required for calls to
 * audit_log_*format.  If the tsk is a task that is currently in a
 * syscall, then the syscall is marked as auditable and an audit record
 * will be written at syscall exit.  If there is no associated task, tsk
 * should be NULL. */
struct audit_buffer *audit_log_start(struct audit_context *ctx, int type)
{


What does  refer to in the function description?
There is no  in this function.

Thanks,
---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/