BIND 10 #183: UNIX domain socket for msgq

[BIND 10 Development]

#183: UNIX domain socket for msgq
-----------------------------+----------------------------------------------
 Reporter:  larissas         |        Owner:  jinmei                     
     Type:  defect           |       Status:  reviewing                  
 Priority:  major            |    Milestone:  04. 2nd Incremental Release
Component:  message-library  |   Resolution:                             
 Keywords:                   |    Sensitive:  0                          
-----------------------------+----------------------------------------------

Comment(by jinmei):

 Replying to [comment:10 jelte]:
 > Ok i have addressed almost all of your issues, except two, which I'm not
 sure about, see below. But first some comments about what i did in r1933;
 >
 > - Those pass statements shouldn't have been there, nor should the
 commented-out lines, so i removed them.
 > - I have added a session_unittest file, which right now *only* tests for
 that big filename exception (which is now thrown instead of cutting the
 file), i think we should make a seperate task for writing tests for this
 (since there didn't appear to be any, and most of the other tests are imho
 out of scope for this ticket).
 > - Re-add the environment variable again (and added a try-catch block to
 the tests, if prefix/install dirs aren't available the test would have
 failed

 So far, Looks okay (modulo r1940).

 > For the not-larger-than-sun_path check, do you think we should make the
 test so that it checks for the boundary (which differs per system) or
 leave it like this, at something that is certainly too big?

 I'd check the boundary, both the okay and NG cases (although I can live
 with "reasonably too long" a name if that's difficult).  As for the per-
 system difference, can't we use {{{sizeof(sockaddr_un.sun_path)}}}?

 now looking at "the other two"...

-- 
Ticket URL: <https://bind10.isc.org/ticket/183#comment:11>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development