[PATCH] Make more PIDL-generated tables static const

Andrew Bartlett abartlet at samba.org
Mon Nov 28 18:15:46 UTC 2016


On Mon, 2016-11-28 at 10:23 +0100, Andreas Schneider wrote:
> On Monday, 28 November 2016 16:55:13 CET Andrew Bartlett wrote:
> > 
> > On Mon, 2016-11-21 at 17:04 +1300, Andrew Bartlett wrote:
> > > 
> > > While working on the previous patches, I noticed some function
> > > tables
> > > generated by PIDL are not static const.  This patches makes them
> > > static
> > >  and or const, which moves them to single-file scope read-only
> > > memory.
> > > 
> > > Please comment and push if OK.
> > 
> > Can I please get a review on these?
> 
> The patches look good, but:
> 
> 
> git am3 ~/Downloads/p/0020-s4-rpc_server-Avoid-extern-reference-to-
> dcesrv_mgmt_.patch
> Applying: s4-rpc_server: Avoid extern reference to
> dcesrv_mgmt_interface and 
> memcpy()
> fatal: sha1 information is lacking or useless (source4/rpc_server/
> dcerpc_server.c).
> error: could not build fake ancestor
> Patch failed at 0001 s4-rpc_server: Avoid extern reference to 
> dcesrv_mgmt_interface and memcpy()
> The copy of the patch that failed is found in: .git/rebase-
> apply/patch
> When you have resolved this problem, run "git am --continue".
> If you prefer to skip this patch, run "git am --skip" instead.
> To restore the original branch and stop patching, run "git am --
> abort".
> 
> 
> Please rebase on master and push yourself. RB+

Thanks.  These are as you have probably seen a part of my larger multi-
process patch set I re-sent to the list yesterday.  I'll get them
separated or also get reviewed what it depends on.

Thanks!

Andrew Bartlett

-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba




More information about the samba-technical mailing list